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

Add patient notes pop-up on consultation page #5847

Closed
wants to merge 37 commits into from

Conversation

Bhavik-ag
Copy link
Contributor

@Bhavik-ag Bhavik-ag commented Jul 7, 2023

WHAT

🤖 Generated by Copilot at 5356861

This pull request adds a new feature to the consultation details page that allows users to view and add notes for a patient in a facility. It introduces a new component PatientNotesSlideover that renders a slideover panel for the patient notes and integrates it with the existing ConsultationDetails component. It also adds several new components and dependencies to support the infinite scrolling and rendering of the notes list and the note card.

Proposed Changes

Backend Changes

Screenshots
image
image
image

@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 5356861

  • Add a slideover panel for viewing and adding notes for a patient in a facility (link, link, link)
  • Add a list of notes for a patient in a facility using infinite scrolling (link, link)
  • Add a card for displaying the information of a single note for a patient in a facility (link)

@Bhavik-ag Bhavik-ag requested a review from a team July 7, 2023 05:59
@Bhavik-ag Bhavik-ag requested a review from a team as a code owner July 7, 2023 05:59
@Bhavik-ag Bhavik-ag requested a review from bodhish July 7, 2023 05:59
@vercel
Copy link

vercel bot commented Jul 7, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
care-storybook ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 5, 2023 2:58am

@netlify
Copy link

netlify bot commented Jul 7, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit db8b91c
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/64f699453fbadb0008bb99fb
😎 Deploy Preview https://deploy-preview-5847--care-egov-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Bhavik-ag Bhavik-ag marked this pull request as draft July 7, 2023 06:03
@Bhavik-ag Bhavik-ag marked this pull request as ready for review July 7, 2023 07:59
@Bhavik-ag
Copy link
Contributor Author

Bhavik-ag commented Jul 8, 2023

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Jul 14, 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.

@netlify
Copy link

netlify bot commented Jul 19, 2023

Deploy Preview for care-net failed.

Name Link
🔨 Latest commit d59542a
🔍 Latest deploy log https://app.netlify.com/sites/care-net/deploys/64b764d0293df90008f74283

@github-actions github-actions bot added the Deploy-Failed Deplyment is not showing preview label Jul 19, 2023
@rithviknishad rithviknishad removed Deploy-Failed Deplyment is not showing preview merge conflict pull requests with merge conflict labels Jul 19, 2023
@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Aug 22, 2023
@Bhavik-ag
Copy link
Contributor Author

All these changes are done

  • doctor status and role to be stored as static attribute
  • make the chat border of remote specialist to primary colour
  • Add a space between "DistrictAdmin" user id
  • Add a "doctor" role type to the home facility doctor

@nihal467 PR is ready for final testing !!

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Sep 11, 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

we are already have a PR raised for the issue #6224

@nihal467 nihal467 closed this Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cypress passed merge conflict pull requests with merge conflict needs testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redesign Doctor Notes as Chat Popup
4 participants