-
-
Notifications
You must be signed in to change notification settings - Fork 62
adding error message to form app questionnaire #806
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
…hold templates with add additional application routes and enhance questionnaire and household templates
…hold templates with improved tooltips and validation
…cy across templates
…calStorage" This reverts commit 9a76637. revert change
…ute in main_site.py
Created Local Storage to Save User Progress
…le and tablet screens
… screens to increase readability
…for improved visibility
…tml to improve readability
Renamed validateForm() to validateHouseholdForm() to avoid function name conflict with other scripts.
…s://github.com/Techtonica/techtonica.org into feature/move-household-script-to-external-file
[Issue 688] Refactor css app reference page
…-to-external-file Feature/move household script to external file
Refactor app-reference.html to move <script> block into external Java…
…ainer when user selects any option An event listener was added to the radio buttons so that when the user selects any option, the red border is automatically removed from the radio group container
[Issue 683] Refactor css app-form-details page
… duplicates - Implemented usage of household.js functions in household.html - Removed all duplicate functions from household.html
…iggering [Issue 717] BugFix: Application Form /app-household.html Validation Errors Not Triggering Correctly
…s-to-external-file Move Inline JS in long-text.html to External File
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Vicwin13 Please update this PR's based branch to be mvp and update your feature branch with mvp
. At present this PR is adding in additional work that is actually present in the mvp branch, making much of this PR out of scope for the changes requested. Thank you.
Please also be sure to push up your changes, as is I don't see any new commit history from you.
![]() this is what I get @daaimah123 |
@Vicwin13 that error is saying that this PR does not contain any new work that is different from the Please push up your changes that were implemented onto your feature branch and then you will be able to change the base branch. |
📝 Description
Triggering of error messages to fields that are not selected or filled.
🔂 Changes Made
changes made in the internal script file
⚙️ Related Issue
/app-questionnaire.html
Form Validation Errors Not Triggering Correctly #719🍏 Type of Change
🧰 New Environment Variables or Requirements
none
✅ Checklist