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

Enhanced UI for Principle diagnosis #6261

Closed
wants to merge 2 commits into from

Conversation

Pranshu1902
Copy link
Contributor

@Pranshu1902 Pranshu1902 commented Sep 11, 2023

WHAT

🤖 Generated by Copilot at 8cba965

Added a feature to exclude facilities assigned to a user in FacilitySelect and ManageUsers, and improved the UI of diagnoses in ConsultationDetails.

Proposed 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 8cba965

  • Add exclude_user prop to FacilitySelect component to filter out facilities assigned to a specific user (link, link, link, link)
  • Change text color of diagnosis label in ConsultationDetails component to match design (link)
  • Filter out principal diagnosis from provisional and final diagnoses in ConsultationDetails component to avoid duplication (link, link)

@Pranshu1902 Pranshu1902 requested a review from a team September 11, 2023 09:50
@Pranshu1902 Pranshu1902 requested a review from a team as a code owner September 11, 2023 09:50
@vercel
Copy link

vercel bot commented Sep 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 Sep 11, 2023 2:31pm

@netlify
Copy link

netlify bot commented Sep 11, 2023

Deploy Preview for care-egov-staging ready!

Name Link
🔨 Latest commit 9f028bf
🔍 Latest deploy log https://app.netlify.com/sites/care-egov-staging/deploys/64ff24aeb1df03000828f330
😎 Deploy Preview https://deploy-preview-6261--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.

@nihal467
Copy link
Member

@Pranshu1902
image

in the issue, it was requested to explore ideas to highlight principle diagnosis from the already reflected final and provisional diagnosis, not to simply hide it.

@Pranshu1902
Copy link
Contributor Author

Conflicts with base branch, lost updates

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better UI for Principle diagnosis
2 participants