Skip to content
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

[filter_box] allow empty filters list #7220

Merged
merged 1 commit into from
Apr 5, 2019

Conversation

mistercrunch
Copy link
Member

in some cases, people want a time filter only on filter box, without
specifying dimensions (filters), this allows that

in some cases, people want a time filter only on filter box, without
specifying dimensions (filters), this allows that
@codecov-io
Copy link

Codecov Report

Merging #7220 into lyftga will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           lyftga    #7220   +/-   ##
=======================================
  Coverage   64.43%   64.43%           
=======================================
  Files         424      424           
  Lines       20759    20759           
  Branches     2277     2277           
=======================================
  Hits        13376    13376           
  Misses       7257     7257           
  Partials      126      126
Impacted Files Coverage Δ
superset/assets/src/explore/controls.jsx 42.4% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1132c3c...60dd3ae. Read the comment docs.

Copy link
Contributor

@xtinec xtinec left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@xtinec
Copy link
Contributor

xtinec commented Apr 4, 2019

This should close #6860? @mistercrunch

@vylc vylc added the lyft Related to Lyft label Apr 4, 2019
@xtinec xtinec merged commit e39b169 into apache:lyftga Apr 5, 2019
michellethomas pushed a commit to michellethomas/panoramix that referenced this pull request Apr 8, 2019
in some cases, people want a time filter only on filter box, without
specifying dimensions (filters), this allows that

(cherry picked from commit e39b169)
michellethomas added a commit that referenced this pull request Apr 8, 2019
in some cases, people want a time filter only on filter box, without
specifying dimensions (filters), this allows that

(cherry picked from commit e39b169)
michellethomas added a commit that referenced this pull request Apr 8, 2019
in some cases, people want a time filter only on filter box, without
specifying dimensions (filters), this allows that

(cherry picked from commit e39b169)
(cherry picked from commit 5ef2712)
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.34.0 labels Feb 28, 2024
cccs-RyanK added a commit to CybercentreCanada/superset that referenced this pull request Jun 4, 2024
…ome cases, people want a time filter only on filter box, withoutspecifying dimensions (filters), this allows that(cherry picked from commit e39b169)
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 lyft Related to Lyft 🚢 0.34.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants