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

Year Of Birth Field Validation Condition in Patient transfer form is not working as expected #9132

Closed
nihal467 opened this issue Nov 15, 2024 · 2 comments · Fixed by #9133
Closed
Assignees

Comments

@nihal467
Copy link
Member

Describe the bug

The "Year of Birth" field in the Patient Transfer form triggers an error check with each user input. This behavior is not aligned with the expected workflow.

To Reproduce
Steps to reproduce the behavior:

  1. Navigate to the Patient tab.
  2. Click on Add Patient.
  3. Enter +919999999999 in the Phone Number field.
  4. In the transfer pop-up that appears, click the green button.
  5. In the next pop-up, enter the Year of Birth.
  6. Observe the validation error triggered with each keystroke.

Expected behavior

Validation should only occur when the input field is out of focus or after the user has completed their entry.

Screenshots

Image

@Muppasanipraneeth
Copy link

can i work on this

@AdityaJ2305
Copy link
Contributor

AdityaJ2305 commented Nov 15, 2024

Hey @Muppasanipraneeth, apologies for missing your comment earlier. I’ve already started working on this issue and pushed a PR for it. Feel free to take up another one.
Thanks!

@github-actions github-actions bot added needs-triage question Further information is requested labels Nov 15, 2024
@Jacobjeevan Jacobjeevan removed question Further information is requested needs-triage labels Nov 19, 2024
@nihal467 nihal467 added this to Care Nov 20, 2024
@nihal467 nihal467 moved this to Done in Care Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants