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 edit option for doctor notes #6919

Merged
merged 9 commits into from
Feb 14, 2024
Merged

Add edit option for doctor notes #6919

merged 9 commits into from
Feb 14, 2024

Conversation

Ashesh3
Copy link
Member

@Ashesh3 Ashesh3 commented Dec 26, 2023

This pull request adds an edit option for doctor notes in the patient consultation notes list. It also includes necessary changes to the API endpoints and models.

Waiting for ohcnetwork/care#1221

@Ashesh3 Ashesh3 requested a review from a team as a code owner December 26, 2023 09:41
Copy link

netlify bot commented Dec 26, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 8650c24
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/65cb63593da04b0008995813
😎 Deploy Preview https://deploy-preview-6919--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.

Copy link

vercel bot commented Dec 26, 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 Feb 13, 2024 0:41am

Copy link

cypress bot commented Dec 26, 2023

Passing run #2084 ↗︎

0 104 0 0 Flakiness 0

Details:

Add edit option for doctor notes
Project: CARE Commit: 8650c240ba
Status: Passed Duration: 03:21 💡
Started: Feb 13, 2024 12:45 PM Ended: Feb 13, 2024 12:48 PM

Review all test suite changes for PR #6919 ↗︎

Copy link

👋 Hi, @Ashesh3,
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.

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Dec 28, 2023
@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Jan 18, 2024
@nihal467
Copy link
Member

@Ashesh3 fix the backend test

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Feb 7, 2024
Copy link

github-actions bot commented Feb 7, 2024

👋 Hi, @Ashesh3,
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

  • the edit button is not clickable , only the icon is clickable

@Ashesh3
Copy link
Member Author

Ashesh3 commented Feb 13, 2024

  • the edit button is not clickable , only the icon is clickable

Fixed, made the edit experience more intuitive.

@nihal467
Copy link
Member

LGTM

@nihal467 nihal467 added tested reviewed reviewed by a core member and removed needs testing labels Feb 13, 2024
@khavinshankar khavinshankar merged commit 3d0e6f2 into develop Feb 14, 2024
61 of 63 checks passed
@khavinshankar khavinshankar deleted the doc-note-edit branch February 14, 2024 07:26
Copy link

@Ashesh3 We truly appreciate your efforts. Thank you for taking the time to contribute; this is a very valuable contribution to us 🥇. We always welcome your contribution 🙂, so feel free to contribute to anything anytime, and never lose that spirit of innovation 🙌.

@Ashesh3 Ashesh3 mentioned this pull request Feb 14, 2024
3 tasks
@aparnacoronasafe
Copy link
Member

@Ashesh3 There is a risk in allowing notes to be edited anytime. Doctor's notes serve as a log of interactions and recommendations between clinicians on a case.

What do you think of limiting such edit option for 30Min? for the doctor to correct any errors in the messages.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewed reviewed by a core member tested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants