Fix Bazza UI data table filters issues#59
Conversation
1. Fix server-side filtering to properly use filters from URL params 2. Ensure filters persist on page refresh by using URL as source of truth 3. Fix pagination to allow navigation between pages 4. Update faceted counts to reflect filtered data instead of original data
|
|
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Join our Discord community for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
📝 Storybook Preview: View Storybook This preview will be updated automatically when you push new changes to this PR.
|
This PR fixes the issues with the Bazza UI data table filters component:
Changes
Fixed server-side filtering with URL params
useFilterSyncwith proper configurationEnsured filters persist on page refresh
Fixed pagination to allow navigation between pages
Improved faceted counts calculation
calculateFacetedCountsto use filtered data instead of original dataTesting
The component now works as expected:
These changes follow the Bazza UI v0.2 model for server-side filtering as documented in the changelog.
💻 View my work • About Codegen