-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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(native-filters): Add time filters #12946
feat(native-filters): Add time filters #12946
Conversation
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 round of comments
Codecov Report
@@ Coverage Diff @@
## master #12946 +/- ##
==========================================
- Coverage 69.14% 62.09% -7.05%
==========================================
Files 1025 972 -53
Lines 48765 46050 -2715
Branches 5188 4485 -703
==========================================
- Hits 33718 28596 -5122
- Misses 14913 17454 +2541
+ Partials 134 0 -134
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Currently the time range filter in FilterBox is not tied to any datasource. It will apply to any chart that is in scope and has a time column specified, regardless if it's using the same datasource or not. The new time filter in native filters here seems to be tied to a specific datasource and field as users have to specify that when creating one, I'm wondering how this actually works in the dashboard? Will it work like the old Time Range filter in If it's the latter case, we might want to redesign the whole thing to make users select filter type first before selecting datasource ("Datasource" and "Field" should be hidden when they select "Filter type: Time"). |
This will be changed shortly - the time filter won't be tied to a datasource, and will only have the time picker (no need for datasource, column etc). Also, the date column filter will be ported to native filters, and that one will obviously have a datasource.
The redesign is currently under discussion, and a change similar to the one you're proposing is in the works |
superset-frontend/src/explore/components/controls/DateFilterControl/DateFilterControl.tsx
Outdated
Show resolved
Hide resolved
Closing in favor of #12992 (includes these commits + follow-up commits) |
SUMMARY
The aim of this PR was to create foundations for new native filters' feature: time filters.
User can choose between one more filter type, which is Time.
That's the basics for this feature. What should be added in next PRs:
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
Go to
config.py
and set"DASHBOARD_NATIVE_FILTERS": True,
Go to dashboard and create native filters with filter type "TIME"
ADDITIONAL INFORMATION
cc @villebro @simcha90 @junlincc