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 medico-legal case toggle and filter #6440

Merged
merged 8 commits into from
Oct 25, 2023
Merged

Add medico-legal case toggle and filter #6440

merged 8 commits into from
Oct 25, 2023

Conversation

Ashesh3
Copy link
Member

@Ashesh3 Ashesh3 commented Oct 11, 2023

Backend PR: ohcnetwork/care#1669

WHAT

🤖 Generated by Copilot at 38c0d4d

This pull request adds a new feature to filter and update patients by their medico-legal status, which is a boolean flag indicating whether their last consultation was a legal case or not. It also improves the SwitchTabs component by renaming its props and using a boolean value for the active tab. The changes affect the PatientManager, PatientFilter, PatientInfoCard, ResourceBoardView, and BoardView components, as well as the ConsultationModel interface and the partial update consultation API route.

Proposed Changes

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

HOW

🤖 Generated by Copilot at 38c0d4d

  • Rename props of SwitchTabs component to use camelCase and boolean values (link, link, link, link, link, link, link)
  • Add medico_legal_case property to ConsultationModel interface to store the medico-legal status of the consultation (link, link)
  • Add last_consultation_medico_legal_case property to PatientManager component to filter patients by medico-legal status (link, link, link)
  • Add last_consultation_medico_legal_case property to PatientFilter component to store and pass the filter value (link, link, link, link)
  • Add SelectMenuV2 component for Is Medico-Legal Case filter to PatientFilter component to allow user to choose filter option (link)
  • Add SwitchTabs component for Is Medico-Legal Case toggle to PatientInfoCard component to allow user to change medico-legal status of consultation (link, link, link, link)
  • Add switchMedicoLegalCase function to PatientInfoCard component to make API call to update medico-legal status of consultation and handle responses (link)
  • Add TRes and TBody types to partial update consultation API route in routes module to specify expected response and request body types (link)

@Ashesh3 Ashesh3 requested a review from a team October 11, 2023 07:38
@Ashesh3 Ashesh3 requested a review from a team as a code owner October 11, 2023 07:38
@Ashesh3 Ashesh3 requested a review from gigincg October 11, 2023 07:38
@vercel
Copy link

vercel bot commented Oct 11, 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 Oct 25, 2023 2:15pm

@netlify
Copy link

netlify bot commented Oct 11, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 30ec443
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/653922c477a6670009c5d400
😎 Deploy Preview https://deploy-preview-6440--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.

@cypress
Copy link

cypress bot commented Oct 11, 2023

Passing run #1469 ↗︎

0 85 0 0 Flakiness 0

Details:

Add medico-legal case toggle and filter
Project: CARE Commit: 30ec443e6b
Status: Passed Duration: 03:13 💡
Started: Oct 25, 2023 2:18 PM Ended: Oct 25, 2023 2:21 PM

Review all test suite changes for PR #6440 ↗︎

@nihal467
Copy link
Member

UI Approved by @aparnacoronasafe

@nihal467
Copy link
Member

LGTM the functionality have few thoughts in the UI

@nihal467
Copy link
Member

@Ashesh3 are the UI changes implemented

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

@Ashesh3
Copy link
Member Author

Ashesh3 commented Oct 25, 2023

@Ashesh3 are the UI changes implemented

Yes, Implemented.

@nihal467
Copy link
Member

nihal467 commented Oct 25, 2023

@Ashesh3
image

  • link abha number UI is fully black, keep it consistent like other icons
  • resolve the merge conflict
  • get the UI opinion from @gigincg

@github-actions github-actions bot removed the merge conflict pull requests with merge conflict label Oct 25, 2023
@Ashesh3
Copy link
Member Author

Ashesh3 commented Oct 25, 2023

@Ashesh3 image

  • link abha number UI is fully black, keep it consistent like other icons
  • resolve the merge conflict
  • get the UI opinion from @gigincg

Resolved and design approved.
image

@nihal467
Copy link
Member

LGTM

@github-actions github-actions bot added the merge conflict pull requests with merge conflict label Oct 25, 2023
@github-actions
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 removed the merge conflict pull requests with merge conflict label Oct 25, 2023
@khavinshankar khavinshankar merged commit f6d31f0 into develop Oct 25, 2023
30 checks passed
@github-actions
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 🙌.

@khavinshankar khavinshankar deleted the issue-6367 branch October 25, 2023 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Medico-Legal Case
3 participants