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

Basic Support for Offline Registered Patients #527

Conversation

manuelroemer
Copy link
Member

@manuelroemer manuelroemer commented Jan 27, 2022

Requirements

  • This PR has a title that briefly describes the work done, including the ticket number if there is a ticket.
  • My work conforms to the OpenMRS 3.0 Styleguide.
  • I checked for feature overlap with existing widgets.

Summary

Provides the groundwork for offline registered patient support within the patient chart: The chart is able to display offline registered patient and pass along the patient's data to the patient chart widgets. The patient banner displays the patient's data and users are able to fill offline forms for that patient.

Screenshots

None.

Issue

https://issues.openmrs.org/browse/O3-981
https://issues.openmrs.org/browse/O3-948
https://issues.openmrs.org/browse/O3-1005
https://issues.openmrs.org/browse/O3-1021
https://issues.openmrs.org/browse/O3-642

Other

Should ideally be merged together with openmrs/openmrs-esm-patient-management#86 as these are dependent on each other.

… into feature/offline-registered-patient-chart
… into feature/offline-registered-patient-chart
… into feature/offline-registered-patient-chart
… into feature/offline-registered-patient-chart
@FlorianRappl FlorianRappl merged commit 9782f78 into openmrs:master Jan 27, 2022
@@ -112,6 +112,7 @@ export async function setupOfflineEncounterSync() {
async function queueEncounterRequest(item: QueuedEncounterRequest) {
const descriptor: QueueItemDescriptor = {
id: item.body.uuid,
displayName: 'Patient form',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be translated?

};

const descriptor: QueueItemDescriptor = {
id: offlineVisit.uuid,
dependencies: [],
displayName: 'Offline visit',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be translated?

encounters: [],
visitType: {
uuid: offlineVisit.visitType,
display: 'OFFLINE_VISIT_PLACEHOLDER',
display: 'Offline',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be translated?

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.

3 participants