-
Notifications
You must be signed in to change notification settings - Fork 347
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
Fixed malfunction of filters #5362
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
✅ Deploy Preview for care-egov-staging ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome! 🚀 LGTM
SonarCloud Quality Gate failed.
|
LGTM |
WHAT
🤖 Generated by Copilot at 58bab83
This pull request adds or improves filtering and ordering features for various data types, such as assets, shifting, resources, samples, patients, and users. It also fixes a bug where the filter values were not reset when the query parameters changed. It does so by adding a
key
prop to the filter components, which is set to the current window location search string, and by using constants and functions to map the query values to more descriptive texts. The changes affect several files, such asAssetsList.tsx
,BadgesList.tsx
,FacilityFilter.tsx
,ListFilter.tsx
, andUserFilter.tsx
.Proposed Changes
Fixed malfunction of filters
@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist
HOW
🤖 Generated by Copilot at 58bab83
key
prop to various filter components to ensure they are re-rendered when the query parameters change and the filter values are reset accordingly (link, link, link, link, link, link, link, link, link, link)facilityName
to store the fetched name of the selected facility in theAssetsList
component and use it to display the facility name badge (link, link, link)getDescShiftingFilterOrder
to display the ordering filter badge for shifting data with a more descriptive text instead of the query parameter value, using theSHIFTING_FILTER_ORDER
constant (link, link, link, link, link)BadgesList
component (link)