feat: add global search across streams - #933
Conversation
- New GlobalSearch component in header: debounced (300ms) search across stream ID, sender, recipient, and asset code via existing listStreams q param - Dropdown shows matching results with sender/recipient/amount preview - Full keyboard support: arrow up/down to navigate, Enter to open stream detail, Escape to close - Enter/click navigates to dashboard with ?streamId= set, reusing the existing useUrlFilters mechanism that drives StreamDetailDrawer Includes same environment fixes applied on other branches tonight (package.json/lock restore, react-router-dom dependency, pre-existing SenderDashboard syntax error, stale story type mismatches) since none have landed on main yet.
|
@Ademiitura is attempting to deploy a commit to the ritik4ever's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
@Ademiitura Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Commit Type
Breaking change? (If yes, add
BREAKING CHANGEin the commit footer)What changed
Testing done
npm run dev) and confirmed via the Network tab that typing into the search box fires a correctly debouncedstreams?q=...&limit=8request only after ~300ms of no typing, with the right query value at each stage.npx tsc --noEmitpasses cleanly.npm run lintshows no new errors introduced by this PR (only pre-existing errors in unrelated files remain).npm cisucceeds against the regenerated lock file.Related issues
Closes #750
Checklist