Added consent indicators and filters

[skks1212]

Dependent on coronasafe/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	Jun 24, 2024 5:51pm

[netlify]

✅ Deploy Preview for care-egov-staging ready!

Name	Link
🔨 Latest commit	709cc8c
🔍 Latest deploy log	https://app.netlify.com/sites/care-egov-staging/deploys/6679b1cd8446eb00085c5052
😎 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 #2796 ↗︎

0

126

0

0

0

Details:

Added consent indicators and filters
Project: CARE	Commit: 709cc8ce2d
Status: Passed	Duration: 03:36 💡
Started: Jun 24, 2024 5:54 PM	Ended: Jun 24, 2024 5:58 PM

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

[skks1212]

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

[skks1212]

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