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

[dashboard fix]: Fix default_filters validation #5297

Merged

Conversation

graceguo-supercat
Copy link

@graceguo-supercat graceguo-supercat commented Jun 27, 2018

if user update dashboard default_filters metadata with invalid slice_id (a slice not included in the dashboard), we should not add this slice id into dashboard metadata. This PR is to fix error dictionary changed size during iteration in python3

@williaster @john-bodley @mistercrunch

@codecov-io
Copy link

Codecov Report

Merging #5297 into master will increase coverage by <.01%.
The diff coverage is 60%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #5297      +/-   ##
=========================================
+ Coverage   61.29%   61.3%   +<.01%     
=========================================
  Files         368     368              
  Lines       23457   23457              
  Branches     2716    2716              
=========================================
+ Hits        14379   14381       +2     
+ Misses       9066    9064       -2     
  Partials       12      12
Impacted Files Coverage Δ
superset/views/core.py 72.96% <60%> (+0.14%) ⬆️

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 4ee984c...d527f4e. Read the comment docs.

Copy link
Contributor

@williaster williaster left a comment

Choose a reason for hiding this comment

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

LGTM 🔥

@graceguo-supercat graceguo-supercat merged commit 17b4298 into apache:master Jun 27, 2018
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this pull request Jun 27, 2018
@graceguo-supercat graceguo-supercat deleted the gg-FixRemoveInvalidSliceId branch July 2, 2018 19:04
hughhhh pushed a commit to lyft/incubator-superset that referenced this pull request Jul 11, 2018
hughhhh pushed a commit to lyft/incubator-superset that referenced this pull request Jul 11, 2018
timifasubaa pushed a commit to airbnb/superset-fork that referenced this pull request Jul 25, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Jul 26, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Jul 26, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Jul 26, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Jul 27, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Jul 31, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Aug 3, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Aug 3, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Aug 4, 2018
mistercrunch pushed a commit to lyft/incubator-superset that referenced this pull request Aug 4, 2018
wenchma pushed a commit to wenchma/incubator-superset that referenced this pull request Nov 16, 2018
@mistercrunch mistercrunch added 🍒 0.27.0 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.28.0 labels Feb 27, 2024
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 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 🍒 0.27.0 🚢 0.28.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants