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

Consultation Specific route for Doctor Notes #6818

Closed
3 tasks
rithviknishad opened this issue Dec 8, 2023 · 1 comment · Fixed by #6851
Closed
3 tasks

Consultation Specific route for Doctor Notes #6818

rithviknishad opened this issue Dec 8, 2023 · 1 comment · Fixed by #6851
Assignees

Comments

@rithviknishad
Copy link
Member

rithviknishad commented Dec 8, 2023

Describe the bug
ohcnetwork/care@1ce8fdc added support for doctor notes to be consultation-specific. However, the front-end still shows notes from all consultations.

To Reproduce
Steps to reproduce the behavior:

  1. Go to https://care.ohc.network/facility/0c95c7f0-e1d2-4aff-83fa-933cef60d3a8/patient/7e8fc20f-02da-47f7-8c33-f2d8c7e52dd8/notes
  2. See error

Tasks

  • Add doctor notes route to be consultation specific. Currently, it is of: /facility/id/patient/id/notes. It should be /facility/id/patient/id/consultation/id/notes
  • Doctor notes from that route should show doctor notes only for that consultation.
  • However, Patient Notes opened from Patient Details page should show all notes, this can retain the current route itself instead. (Maybe we could indicate which consultation the note belongs to in this view, maybe as a chip showing the consultation admission date + facility name or something similar which when clicked would take you to the consultation details)
@sainak
Copy link
Member

sainak commented Dec 12, 2023

Some notes don't have a consultation linked to them
So we should display all notes for a patient I. The patient details page

cc @Ashesh3 @rithviknishad

@rithviknishad rithviknishad moved this from Triage to Up Next in Care Dec 12, 2023
@github-project-automation github-project-automation bot moved this from Up Next to Done in Care Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants