-
Notifications
You must be signed in to change notification settings - Fork 241
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
(feat) O3-4328: Ward App: Don't allow admissions/transfers across vi… #1434
Conversation
This will need the following feature PR to the FHIR2 module to be merged in: |
Size Change: +56 B (0%) Total Size: 7.27 MB ℹ️ View Unchanged
|
@@ -33,8 +36,11 @@ export default function LocationSelector({ paginationSize = 15, ...props }: Loca | |||
if (emrConfiguration) { | |||
criteria.push(['_tag', emrConfiguration.supportsTransferLocationTag.name]); | |||
} | |||
if (ancestorLocation) { | |||
criteria.push(['below', ancestorLocation.uuid]); |
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.
Haven't followed that closely in https://openmrs.atlassian.net/browse/FM2-653, is it supposed to be partof:below
?
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.
I implemented it as just "below" but am looking into getting "partof:below" to work based on Ian's feedback, and then will update this.
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.
One small comment.. You can change the criteria name as needed. The rest looks good.
Updated this with the correct "part:below" terminology. Waiting on FHIR changes to merge this in. |
…sit locations
Requirements
Summary
See Jira link
Screenshots
Related Issue
https://openmrs.atlassian.net/browse/O3-4328
Other