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

Date and Time component restricts selecting same-day discharge date for patients due to non-proper future date validation. #9011

Closed
2 tasks
nihal467 opened this issue Nov 6, 2024 · 8 comments · Fixed by #9014
Assignees
Labels
P1 breaking issue or vital feature urgent

Comments

@nihal467
Copy link
Member

nihal467 commented Nov 6, 2024

Describe the bug

  • When discharging a patient on the same day as their admission, the discharge date selector does not allow selecting the admission date in the patient discharge pop-up.

To Reproduce
Steps to reproduce the behavior:

  1. Go to the patient tab
  2. Select any active patient to open their dashboard.
  3. Click on Manage Patient and choose Edit Consultation.
  4. Set the patient admission date to today’s date.
  5. Submit the form to return to the patient dashboard.
  6. Click on Manage Patient again and select Discharge from Care.
  7. See error

Expected behavior

  • The date and time component should allow selecting any date and time from the admission date up to the current date and time, supporting same-day discharges.

Screenshots

Image

@nihal467 nihal467 added urgent P1 breaking issue or vital feature labels Nov 6, 2024
Copy link

github-actions bot commented Nov 6, 2024

⚠️ Refrain from assigning this issue to yourself if you have another P1 issue assigned that is not yet closed.

@JavidSumra
Copy link
Contributor

Can You please assign this issue to me

@nihal467
Copy link
Member Author

nihal467 commented Nov 6, 2024

@JavidSumra its a priority issue, can you push the fix by EOD

@JavidSumra
Copy link
Contributor

I'll check and let you know

@nihal467 nihal467 assigned JavidSumra and unassigned JavidSumra Nov 6, 2024
@nihal467
Copy link
Member Author

nihal467 commented Nov 6, 2024

@JavidSumra check and comment over here, then i will assign you the issue

@JavidSumra
Copy link
Contributor

@nihal467 I'll work on this issue

@JavidSumra
Copy link
Contributor

Hey @nihal467 can you please assign 2 Story Point for this?
cc @rithviknishad

@rithviknishad
Copy link
Member

@JavidSumra 1 story point is reasonable for small changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 breaking issue or vital feature urgent
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants