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

Issue 730. EHR. Photo ID and insurance cards are duplicated on visit details page #731

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

RomanGolchuk
Copy link
Contributor

@RomanGolchuk RomanGolchuk commented Jan 23, 2025

Comment on lines 1019 to 1020
subject: { reference: `Patient/${patientID}` },
context: { related: [{ reference: `Appointment/${appointmentID}` }, { reference: `Patient/${patientID}` }] },
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is required for some docref files to work correctly in ehr, why was it deleted? For example patient photos to be loaded in telemed visit.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change aligns with the url and comment in the EHR code:
https://github.com/masslight/ottehr/blob/develop/apps/ehr/src/pages/AppointmentPage.tsx#L666-L668

The files prepared here seem should to match that request.
However, I might have misunderstood the context, as this section also includes school and work notes, except for Photo ID and Insurance Cards.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It seems Photo ID and Insurance Cards should also relate to the appointment.

Copy link
Contributor

Choose a reason for hiding this comment

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

Why should photo id and insurance cards be connected to the appointment?

Copy link
Collaborator

Choose a reason for hiding this comment

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

in the visit details of concrete appointment we show data that was provided during this concrete visit paperwork flow, right? So, data is taken from the related QR. Then why should we show docs that are not relevant for this appt that were updated after this visit? @imbenham

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