Added consent indicators and filters

[shivankacker]

Dependent on ohcnetwork/care#2255

Proposed Changes

• Fixes flag on patient live encounter with no consent for admission #7972
• Fixes Improvements to Consent Records #7993

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

Screenshots

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

[vercel]

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	Jul 17, 2024 6:01am

[netlify]

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	bcbe837
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/66975e02391a17000871cfe6
😎 Deploy Preview	https://deploy-preview-8003--care-egov-staging.netlify.app
📱 Preview on mobile	Toggle 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]

Passing run #2880 ↗︎

0

126

0

0

0

Details:

Added consent indicators and filters
Project: CARE	Commit: bcbe8371dd
Status: Passed	Duration: 02:53 💡
Started: Jul 17, 2024 10:25 AM	Ended: Jul 17, 2024 10:28 AM

Review all test suite changes for PR #8003 ↗︎

[aparnacoronasafe]

@skks1212 The message on patient list "16 patients admitted without consent" may be edited to say "16 patients admitted missing consent records"

[nihal467]

@skks1212

• in the badge we mention None, and in the advance filters, the dropdown mention the option as no consents, keep it consistent, in both places

• if a user have multiple facility access and use the advance filter in the patient tab to view only patient list of a particular facility, during this scenario, the missing consent banner count should also filter along with it to just show missing consent of that particular facility.

similar, user apply any advance filter, in the patient tab, it should filter the missing patient consent banner based on that

[shivankacker]

@nihal467, are you sure the filters apart from facility filters should change the number shown in the alerts? I believe it is to show patients in that facility (or everywhere) that have no consents regardless of filters. @aparnacoronasafe can you confirm?

[aparnacoronasafe]

The experience is better when the number on alert responds to all filters.

If user applies a filter with list showing on 2 results but alert on top says "10 records with no consent record" doesn't look very convincing.

Is there challenge to make it respond to all filters?

You are correct in saying that the need to be able to filter based on facility is not valuable in this case.

[shivankacker]

@aparnacoronasafe no challenge, just had a doubt. Made the changes

[khavinshankar]

@skks1212 seems multiple cypress tests are failing, please fix those

[github-actions]

@skks1212 Your efforts have helped advance digital healthcare and TeleICU systems. 🚀 Thank you for taking the time out to make CARE better. We hope you continue to innovate and contribute; your impact is immense! 🙌