Skip to content
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

Improve Age Validation in User Creation Form #9001

Closed
Rishith25 opened this issue Nov 5, 2024 · 6 comments · Fixed by #9080
Closed

Improve Age Validation in User Creation Form #9001

Rishith25 opened this issue Nov 5, 2024 · 6 comments · Fixed by #9080
Assignees

Comments

@Rishith25
Copy link
Contributor

Description:Currently, when creating a user, if the Date of Birth (DOB) entered corresponds to an age less than 15 years, the system makes an API call and returns an error with the message: "Age must be greater than 15 years."

Image

Proposed Improvement: To enhance the user experience and reduce unnecessary API calls, dynamic client-side validation should be added to the form. This validation would check if the age is at least 15 years before submitting the form, thereby providing immediate feedback to the user.

Expected Behavior:

The form should display an error message dynamically if the user enters a DOB indicating an age less than 15 years.
The form should prevent the API call from being made if the age criteria are not met.
Benefits:

Reduces API traffic.
Improves user experience with real-time feedback.

@Rishith25
Copy link
Contributor Author

@rithviknishad I would like to work on this can you assign it to me.

@rithviknishad
Copy link
Member

@Jacobjeevan Could you confirm and assign accordingly if it's being done along with #8878

@Jacobjeevan
Copy link
Contributor

@Jacobjeevan Could you confirm and assign accordingly if it's being done along with #8878

@rithviknishad I wasn't planning on editing this page, but since we are using most of the same fields in single user detail/management page, it just makes sense to reuse those components here 🤔

@Jacobjeevan Jacobjeevan self-assigned this Nov 5, 2024
@Rishith25
Copy link
Contributor Author

@rithviknishad should I proceed with this issue?

@rithviknishad
Copy link
Member

@Jacobjeevan's PR would be addressing this too @Rishith25

@Jacobjeevan
Copy link
Contributor

@abhishekreddy8604 The issue is already assigned and it's taken care of 👍

@Jacobjeevan Jacobjeevan added this to Care Nov 7, 2024
@github-project-automation github-project-automation bot moved this to Triage in Care Nov 7, 2024
@Jacobjeevan Jacobjeevan moved this from Triage to In Progress in Care Nov 7, 2024
@Jacobjeevan Jacobjeevan moved this from In Progress to Review required in Care Nov 15, 2024
@github-project-automation github-project-automation bot moved this from Review required to Done in Care Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants