-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
fix(native-filters): emitted filter label format #16828
fix(native-filters): emitted filter label format #16828
Conversation
Codecov Report
@@ Coverage Diff @@
## master #16828 +/- ##
=======================================
Coverage 76.99% 77.00%
=======================================
Files 1018 1018
Lines 54668 54660 -8
Branches 7456 7452 -4
=======================================
- Hits 42094 42091 -3
+ Misses 12329 12324 -5
Partials 245 245
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/testenv up FEATURE_DASHBOARD_NATIVE_FILTERS=true FEATURE_DASHBOARD_CROSS_FILTERS=true FEATURE_DASHBOARD_NATIVE_FILTERS_SET=true FEATURE_DASHBOARD_FILTERS_EXPERIMENTAL=true |
@villebro Ephemeral environment spinning up at http://54.184.7.171:8080. Credentials are |
LGTM! |
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.
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.
LGTM, Thanks for the fix.
Ephemeral environment shutdown and build artifacts deleted. |
🏷️ 2021.38 |
(cherry picked from commit 0a8d0c6)
SUMMARY
The fix in #16460 didn't address the special case of numeric filter values sometimes being temporal filters. It also introduced a new util for formatting filter values whose functionality was already covered by the much more comprehensive
getDataRecordFormatter
factory function, which also handles time and number formatting. The redundant formatter is removed and replaced bygetDataRecordFormatter
which was already being used for formatting the options in the Select component. This will ensure that the filter indicator displays identical values as the Select component.Closes #16808
AFTER
BEFORE
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION