-
Notifications
You must be signed in to change notification settings - Fork 14.3k
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
feat(filter-set): adding new filters #13515
Conversation
…incubator-superset into filter_bar_actions � Conflicts: � superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterBar.tsx
…_to_data_mask � Conflicts: � superset-frontend/package-lock.json � superset-frontend/package.json
…incubator-superset into filter_bar_actions
…incubator-superset into filter_bar_actions
…incubator-superset into filter_bar_actions
…ctions � Conflicts: � superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterBar.tsx
� Conflicts: � superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterBar.tsx
…esign � Conflicts: � superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterBar.tsx � superset-frontend/src/dashboard/components/nativeFilters/FilterBar/state.ts
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.
First pass comments
superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterBar.tsx
Outdated
Show resolved
Hide resolved
superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterSets/FiltersHeader.tsx
Outdated
Show resolved
Hide resolved
superset-frontend/src/dashboard/components/nativeFilters/FilterBar/FilterSets/utils.ts
Show resolved
Hide resolved
Codecov Report
@@ Coverage Diff @@
## master #13515 +/- ##
==========================================
- Coverage 76.47% 72.86% -3.62%
==========================================
Files 914 616 -298
Lines 46073 21574 -24499
Branches 5589 5600 +11
==========================================
- Hits 35235 15719 -19516
+ Misses 10686 5723 -4963
+ Partials 152 132 -20
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
* refactor(native-filters): move data mask to root reducer * refactor: update rest stuff for dataMask * refactor: add ownCrrentState to explore * fix: fix immer reducer * fix: merge with master * refactor: support explore dataMask * refactor: support explore dataMask * docs: add comment * refactor: remove json stringify * fix: fix failed cases * feat: filter bat buttons start * fix: fix CR notes * fix: fix cascade filters * fix: fix CR notes * refactor: add clear all * fix: fix CR notes * fix: fix CR notes * fix: fix CR notes * feat: buttons in filter bar * lint: update imports * feat: add tabs for filter sets * feat: add buttons to filter set * feat: first phase add filter sets * fix: undo FF * fix: fix CR notes * fix: fix CR notes * lint: fix lint * fix: fix filter bar opening * refactor: continue filter sets
SUMMARY
This PR is first phase in adding filters sets according this design: https://www.figma.com/proto/Z75uj6W6n0A6nfK3Mvawqi/Dashboard-Filters---P1?node-id=960%3A2246&scaling=min-zoom
It includes:
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Screen.Recording.2021-03-08.at.12.59.00.mov
TEST PLAN
ADDITIONAL INFORMATION