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

(chore) set restrictByVisitLocationTag parameter to true #869

Merged
merged 1 commit into from
Nov 22, 2024

Conversation

mogoodrich
Copy link
Member

@mogoodrich mogoodrich commented Nov 22, 2024

This will update the Reference Application to use the newly-implemented functionality to restrict visits to those tagged as visit locations (and to default the visit location to the "nearest" visit location associated with the session location), following the O2 Ref App model.

@ibacher ibacher merged commit f2a62c5 into main Nov 22, 2024
13 checks passed
@mogoodrich
Copy link
Member Author

Thanks @ibacher , I confirmed that this is place and seems to be working (after a ctrl-reload).

@gracepotma lmk if you have any questions around this, but, basically, when starting a visit you can now only choose a location that is tagged as a "Visit Location". In most cases (like with all the "Site x" locations) the locations have been tagged as both Visit and Login locations so you won't see much change. What is different is for the 5 locations that I've made children of "Ubuntu Hospital" (Inpatient Ward, Outpatient Ward, Wards 1, 2 & 3) you won't be able to choose them as visit locations, and if you are logged into those locations and try to start a visit, the default visit location is set to "Ubuntu Hospital" (representing the fact that your overall visit is at Ubuntu Hospital, though you may have encounters at various different locations within it).

NethmiRodrigo pushed a commit to openmrs/path-drc-emr that referenced this pull request Dec 17, 2024
NethmiRodrigo added a commit to openmrs/path-drc-emr that referenced this pull request Dec 17, 2024
* Update the billing concepts to add service types ConvSet (openmrs#857)

Co-authored-by: Amos Laboso <[email protected]>

* Bump attachments to 3.5.0-SNAPSHOT

* (feat) O3-3886: Demo Ward App in the Ref App (openmrs#851)

* Update patient photo concept

* Update stuctured_soap_note.json (openmrs#860)

* (feat) O3-3886: Demo Ward App in the Ref App (openmrs#862)

* update to latest release of EMR API and REST web services

* Bump the legacyui module to 1.19.0-SNAPSHOT

* Switch the legacyui module to version 1.19.0

* Switch the attachments module to version 3.5.0

* Bump the legacyui module to version 1.20.0

* (chore) set restrictByVisitLocationTag parameter to true (openmrs#869)

* Finalize version updates

* (feat) O3-4185: Add form for IPD admission (openmrs#864)

* Bump openmrs core to version 2.6.10-SNAPSHOT

* Bumping the legacyui module to 1.22.0-SNAPSHOT

* Moving global properties in the globalproperties-core_demo.xml into the globalProperties tag (openmrs#872)

* (chore) Add HIV forms and encounter types

* (chore): Update frontend modules (#9)

* (chore): Update logo and favicon (#15)

* (chore): Revert to dev

* (chore): Update translations and forms

* (fix) Revert version updates and fix form logic (#28)

* (fix) Revert version updates

* (fix) Remove patient labs app

* (fix) Fix logic in forms

* (fix) Update global properties tag

* Fixed versions that I messed up in the merge

* Update forms

* Update locations

* Update version of core

---------

Co-authored-by: Amos Laboso <[email protected]>
Co-authored-by: Amos Laboso <[email protected]>
Co-authored-by: dkayiwa <[email protected]>
Co-authored-by: Mark Goodrich <[email protected]>
Co-authored-by: Ian <[email protected]>
Co-authored-by: Wamathaga Kamau <[email protected]>
Co-authored-by: mogoodrich <[email protected]>
Co-authored-by: Vineet Sharma <[email protected]>
Co-authored-by: Wikum Weerakutti <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants