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

fix: dashboard filter value is cleared when 2 similar dashboards opened in succession #21461

Merged
merged 1 commit into from
Sep 14, 2022

Conversation

mayurnewase
Copy link
Contributor

@mayurnewase mayurnewase commented Sep 14, 2022

SUMMARY

fixes: #21353

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before: check the issue description

After:
Screencast from 14-09-22 08:36:50 AM IST.webm

TESTING INSTRUCTIONS

  1. open 1 dashboard add 1 native filter of type value on any column and set any default value and save it
  2. save that dashboard by overwriting itself.
  3. save it again creating a new copy
  4. change the default value for that filter to something else
  5. save this dashboard by overwriting itself.
  6. now go to dashboard list
  7. open original dashboard
  8. should apply default value of its filter

ADDITIONAL INFORMATION

@codecov
Copy link

codecov bot commented Sep 14, 2022

Codecov Report

Merging #21461 (420034e) into master (9c285da) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #21461      +/-   ##
==========================================
+ Coverage   66.53%   66.56%   +0.02%     
==========================================
  Files        1791     1791              
  Lines       68599    68617      +18     
  Branches     7320     7319       -1     
==========================================
+ Hits        45645    45675      +30     
+ Misses      21064    21043      -21     
- Partials     1890     1899       +9     
Flag Coverage Δ
javascript 52.80% <100.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...board/components/nativeFilters/FilterBar/index.tsx 66.18% <100.00%> (+6.04%) ⬆️
...iews/CRUD/data/dataset/AddDataset/Header/index.tsx 87.50% <0.00%> (-12.50%) ⬇️
...ontend/src/explore/controlUtils/getControlState.ts 80.32% <0.00%> (-10.42%) ⬇️
superset-frontend/src/views/CRUD/utils.tsx 64.28% <0.00%> (-0.52%) ⬇️
superset-frontend/src/components/Tooltip/index.tsx 100.00% <0.00%> (ø)
...set-frontend/src/views/CRUD/data/dataset/styles.ts 100.00% <0.00%> (ø)
...et-frontend/src/components/TableSelector/index.tsx 76.00% <0.00%> (ø)
...frontend/src/components/DatabaseSelector/index.tsx 95.23% <0.00%> (ø)
...end/src/components/PageHeaderWithActions/index.tsx 90.00% <0.00%> (ø)
...d/packages/superset-ui-chart-controls/src/types.ts 100.00% <0.00%> (ø)
... and 9 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@michael-s-molina michael-s-molina left a 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.

@michael-s-molina michael-s-molina merged commit 59ca786 into apache:master Sep 14, 2022
michael-s-molina pushed a commit that referenced this pull request Sep 14, 2022
@mayurnewase
Copy link
Contributor Author

don't know how to add test for this.

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 size/XS 🍒 1.5.2 🍒 1.5.3 🚢 2.1.0
Projects
None yet
3 participants