Skip to content

fix(cross-filters): apply cross filters for all charts#13178

Merged
villebro merged 1 commit intoapache:masterfrom
simcha90:fix-cross
Feb 17, 2021
Merged

fix(cross-filters): apply cross filters for all charts#13178
villebro merged 1 commit intoapache:masterfrom
simcha90:fix-cross

Conversation

@simcha90
Copy link
Contributor

SUMMARY

This PR apply cross filter functionality for all charts on dashboard until will be implemented it's scoping

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-02-17.at.17.41.19.mov

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue:
  • Changes UI
  • Requires DB Migration.
  • Confirm DB Migration upgrade and downgrade tested.
  • Introduces new feature or API
  • Removes existing feature or API

@junlincc junlincc added the dashboard:native-filters Related to the native filters of the Dashboard label Feb 17, 2021
@villebro villebro mentioned this pull request Feb 17, 2021
3 tasks
Copy link
Member

@villebro villebro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - Note to self, we need to implement similar NULL formatting in the Select component as we have in the table chart.

@villebro villebro merged commit 4502155 into apache:master Feb 17, 2021
@junlincc junlincc added dashboard:cross-filters Related to the Dashboard cross filters and removed dashboard:native-filters Related to the native filters of the Dashboard labels Apr 27, 2021
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 First shipped in 1.2.0 labels Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels dashboard:cross-filters Related to the Dashboard cross filters size/S 🚢 1.2.0 First shipped in 1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants