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

Add more validation check for patient transfer logic #7662

Open
5 tasks
nihal467 opened this issue Apr 19, 2024 · 2 comments · May be fixed by #7689
Open
5 tasks

Add more validation check for patient transfer logic #7662

nihal467 opened this issue Apr 19, 2024 · 2 comments · May be fixed by #7689

Comments

@nihal467
Copy link
Member

nihal467 commented Apr 19, 2024

Describe the bug

  • When we transfer a patient today to a new facility and create a consultation with admission two days ago, they are being accepted in the system
  • When we transfer a patient to a multiple new facilities, a create a consultation with the admission of the same time, they are being accepted in the system
  • When we transfer a patient today to a new facility, today's date is considered as the discharge date but the user might have forgotten to do this, and they updated the card two days later

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

  • if already a active consultation exist, add a flow to accept manual discharge date from user while patient transfer
  • new facility admission date should be less than the existing discharge date

Screenshots

image

@nihal467 nihal467 added the P1 breaking issue or vital feature label Apr 19, 2024
Copy link

⚠️ Refrain from assigning this issue to yourself if you have another P1 issue assigned that is not yet closed.
@Ashesh3 kindly acknowledge this message by commenting 'Acknowledged' below.

@nihal467
Copy link
Member Author

@aparnacoronasafe can you provide your opinion on this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants