Bug Fixed : Default filters on patients page loads

[mustafaazad03]

WHAT

Solved the bug for An empty filter for emergency phone numbers to be applied by default.

Proposed Changes

• Fixes Patients List: Empty emergency phone number filter applied by default on mount #6101
• Implement the check for phone number and also make it null
• Implement the check for emergency phone number and make it null
• Make the limit equal to null as a default

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

[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	Sep 4, 2023 1:45am

[netlify]

✅ Deploy Preview for care-egov-staging ready!

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

[rithviknishad]

Nice work on fixing it! However, is setting limit=null necessary?

useFilters's limit prop is for identifying the pagination limit when paginating, and is not a default value!

And hence causes to:

Rest all LGTM!

[rithviknishad]

Btw, you can keep pushing to this current branch instead of always re-opening a new PR! The commits will be squashed and merged anyways once tested!

[mustafaazad03]

Ok I will see thanks for pointing out 🙂.

[mustafaazad03]

Hi @rithviknishad, I update all suggested changes will you please check

[rithviknishad]

Nice work @mustafaazad03! LGTM.

[nihal467]

LGTM