ENG-420 feat: add search functionality to FacilitySwitcher and UserDashboard components#16388
ENG-420 feat: add search functionality to FacilitySwitcher and UserDashboard components#16388abhimanyurajeesh wants to merge 2 commits into
Conversation
…shboard components
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
Adds client-side facility search to improve navigation when users have more than four facilities.
Changes:
- Adds facility-name filtering to the sidebar facility switcher.
- Adds facility-name filtering to the Facilities tab on the user dashboard.
- Shows empty/no-results states when searches return no matches.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/pages/UserDashboard.tsx |
Adds search state, filtered facility list rendering, and an empty state for no facility matches. |
src/components/ui/sidebar/facility/facility-switcher.tsx |
Adds a searchable input to the facility dropdown and filters listed facilities. |
🎭 Playwright Test ResultsStatus: ❌ Failed
📊 Detailed results are available in the playwright-final-report artifact. Run: #9106 |
…serDashboard components, add tests for search behavior
Deploying care-preview with
|
| Latest commit: |
e9e5d6b
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://42350d23.care-preview-a7w.pages.dev |
| Branch Preview URL: | https://eng-420.care-preview-a7w.pages.dev |
|
closing in fav of #16248 |
Proposed Changes
Fixes ENG-420
Note: If there is more than 1 facility then only search will be shown.
Tagging: @ohcnetwork/care-fe-code-reviewers
Merge Checklist