-
Notifications
You must be signed in to change notification settings - Fork 290
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
BAH-2647 | Already active Visits not allowed to create again #564
Conversation
atish160384
commented
May 5, 2023
- If the visit of same patient at same location already exists then it is not allowed to create visit again.
@@ -87,6 +87,7 @@ | |||
"MESSAGE_DIALOG_OPTION_COPY" : "Copy Error", | |||
"MESSAGE_DIALOG_OPTION_OKAY": "OK", | |||
"NO_LOCATION_TAGGED_TO_VISIT_LOCATION": "No location tagged to Visit Location Found", | |||
"VISIT OF THIS PATIENT AT SAME LOCATION EXISTS": "Vist of this patient at same location already exists", |
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.
Keys are having spaces! it will be good if spaces are replaced by underscore
This check has been added in the UI codebase, and not in the backend. I think this is incorrect. The UI codebase is just ONE browser client. Ideally this is domain/business logic, which should be in backend, so that once a visit is started by ANY client, no one else should be able to start a visit for the same visit location. I think this code should be in bahmni-core and not in bahmniapps. Also, need to validate with @angshu. Locations can have hierarchy. E.g There is some expectation that one visit can only be started in one sub-tree. But another visit can be started in another sub-tree. For instance the same patient can have parallel visits in OPD and IPD, but only one visit at a time within OPD/IPD. |
We have a different card for Backend fix. https://bahmni.atlassian.net/browse/BAH-1784 |
# Conflicts: # ui/app/i18n/registration/locale_es.json # ui/app/i18n/registration/locale_pt_BR.json
|