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

Redesign Doctor Notes (Review, QA, Testing) #6224

Merged
merged 63 commits into from
Nov 23, 2023

Conversation

Bhavik-ag
Copy link
Contributor

@Bhavik-ag Bhavik-ag commented Sep 5, 2023

WHAT

🤖 Generated by Copilot at 15475e1

This pull request adds a new feature to the consultation module that allows viewing and adding patient notes in a slide-over panel. It introduces new components, constants, and actions for the patient notes feature, and uses a library for infinite scrolling and a websocket for real-time updates. It also modifies the notification module to handle the new event for patient note added. It also makes some minor formatting and readability improvements to some files.

Proposed Changes

  • Testing and QA related PR for Redesign Doctor Notes

Backend Changes

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
  • Update product documentation.
  • Ensure that UI text is kept in I18n files.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
  • Request for Peer Reviews
  • Completion of QA

HOW

🤖 Generated by Copilot at 15475e1

  • Add a slide-over panel for viewing and adding patient notes from the consultation details page (link,link,link,link,link)
  • Implement infinite scrolling for the patient notes list using react-infinite-scroll-component library (link,link)
  • Use PatientNoteCard component to render each patient note as a card with content, date, and creator information (link,link)
  • Use USER_TYPES_MAP constant to display user-friendly labels for the user types of the note creators (link,link)
  • Use useMessageListener hook to listen for updates from the websocket and re-fetch the patient notes accordingly (link,link)
  • Add a new notification event type for PATIENT_NOTE_ADDED and define its text and link (link,link)
  • Refactor the PatientNotes component to use the PatientNotesList component and the TextFormField component (link,link,link,link)
  • Remove line breaks for some icons in UniconPaths.json file (link,link,link)
  • Remove an empty line from Common.json file and add a new line at the end of Consultation.json file (link,link)

Bhavik-ag and others added 30 commits July 7, 2023 11:26
@nihal467
Copy link
Member

the coronasafe/care#1553 backend for making it realtime was rejected,

@nihal467
Copy link
Member

nihal467 commented Oct 25, 2023

@Ashesh3 to work on the co-related backend today, expected delivery by this weekend

CC: @gigincg @khavinshankar

@nihal467
Copy link
Member

@Ashesh3 coronasafe/care#1688 new back-end create

CC: @gigincg @khavinshankar

@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Oct 30, 2023
@github-actions
Copy link

👋 Hi, @Bhavik-ag,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

@nihal467
Copy link
Member

LGTM

@khavinshankar khavinshankar merged commit f804e6d into coronasafe:develop Nov 23, 2023
27 of 36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C4GT Code 4 GovTech cypress passed reviewed reviewed by a core member tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redesign Doctor Notes as Chat Popup [C4GT] Redesign Doctor Notes
5 participants