feat(native-filters): Auto apply changes in FiltersConfigModal

[simcha90]

SUMMARY

1. After user changes filters in FiltersConfig modal and clicks it automatically apply new changes to dashboard without need to click apply

2. Disable controls that affects dataMask in FiltersConfig modal until Default Value populates

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-05-06.at.9.20.12.mov

TEST PLAN

ADDITIONAL INFORMATION

• Has associated issue:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[junlincc]

@simcha90 is this PR only code refactoring? please include video in the PR if there's UI changes 🙏

[simcha90]

@simcha90 is this PR only code refactoring? please include video in the PR if there's UI changes 🙏

done

[villebro]

A few comments - in general many great improvements here!

[codecov]

Codecov Report

Merging #14461 (147d594) into master (68058b0) will increase coverage by 0.17%.
The diff coverage is 92.68%.

@@            Coverage Diff             @@
##           master   #14461      +/-   ##
==========================================
+ Coverage   77.06%   77.24%   +0.17%     
==========================================
  Files         958      958              
  Lines       48242    48261      +19     
  Branches     6062     5649     -413     
==========================================
+ Hits        37179    37277      +98     
+ Misses      10862    10782      -80     
- Partials      201      202       +1     

Flag	Coverage Δ
javascript	72.38% <92.68%> (+0.37%)	⬆️

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

Impacted Files	Coverage Δ
...end/src/filters/components/GroupBy/controlPanel.ts	100.00% <ø> (ø)
...tend/src/filters/components/Select/controlPanel.ts	63.63% <ø> (ø)
...onfigModal/FiltersConfigForm/FiltersConfigForm.tsx	70.17% <75.00%> (-0.07%)	⬇️
superset-frontend/src/dataMask/reducer.ts	70.83% <84.61%> (+2.54%)	⬆️
...et-frontend/src/dashboard/actions/nativeFilters.ts	80.00% <100.00%> (+26.15%)	⬆️
...tersConfigModal/FiltersConfigForm/ControlItems.tsx	100.00% <100.00%> (ø)
...mponents/nativeFilters/FiltersConfigModal/utils.ts	68.75% <100.00%> (+2.51%)	⬆️
superset-frontend/src/dataMask/actions.ts	78.57% <100.00%> (ø)
...tiveFilters/FilterBar/FilterSets/FiltersHeader.tsx	94.44% <0.00%> (+2.77%)	⬆️
...tiveFilters/FilterBar/FilterSets/FilterSetUnit.tsx	100.00% <0.00%> (+3.84%)	⬆️
... and 7 more

---

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 68058b0...147d594. Read the comment docs.

[junlincc]

@simcha90 @villebro Hi both, i would highly appreciate we look at PRs that are tagged "hold" together before merging.

[villebro]

My apologies, this was not intentional, I had missed this label 🙁 I'll look into adding a CI workflow to block merging if any restricted labels are present asap.