refactor(native-filters): Move filtersState to dataMask redux root

[simcha90]

SUMMARY

This PR move (dashboard redux state) -> nativeFilters -> filtersState to redux -> dataMask because of next reasons:

1. it has now not only nativeFilters data but also cross-filters data and own charts data
2. we also use it in explore for reusage
3. it uses dataMask naming instead of filtersState

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2021-03-04.at.11.21.14.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

[villebro]

First pass comments

[villebro]

Second pass - mostly import nits but also a question about whether dataMask should be under src or src/dashboard.

[villebro]

I see the reasoning behind having dataMask on the root level, as they will be used in viz, filters, dashboard and explore, but I still feel superset-frontend/src/dashboard/dataMask might be more appropriate, as that's where these masks are primarily used.

[simcha90]

I think it's the same reason why we put chart in the root, it's parallel to chart I think... Also this feature have enough functionality to have independent place in root :) Also I think in future we will need to add here also ui filtering like ui changes that will applied to other charts, may be also we will need to use here to show how it's applied to itself

[villebro]

FYI @simcha90 this needs a rebase (also left a few comments)

[codecov]

Codecov Report

Merging #13437 (9d80fe1) into master (d0714a0) will decrease coverage by 4.03%.
The diff coverage is 86.17%.

@@            Coverage Diff             @@
##           master   #13437      +/-   ##
==========================================
- Coverage   77.41%   73.37%   -4.04%     
==========================================
  Files         909      611     -298     
  Lines       45863    21336   -24527     
  Branches     5534     5537       +3     
==========================================
- Hits        35503    15656   -19847     
+ Misses      10232     5550    -4682     
- Partials      128      130       +2     

Flag	Coverage Δ
cypress	57.97% <86.25%> (+0.05%)	⬆️
hive	?
javascript	63.26% <50.00%> (-0.02%)	⬇️
mysql	?
postgres	?
presto	?
python	?
sqlite	?

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

Impacted Files	Coverage Δ
superset-frontend/src/chart/ChartContainer.jsx	100.00% <ø> (ø)
...s/nativeFilters/FilterBar/CascadeFilterControl.tsx	85.71% <ø> (ø)
...perset-frontend/src/dashboard/containers/Chart.jsx	100.00% <ø> (ø)
...frontend/src/dashboard/containers/FiltersBadge.tsx	89.28% <ø> (ø)
superset-frontend/src/dashboard/reducers/index.js	100.00% <ø> (ø)
superset-frontend/src/dashboard/reducers/types.ts	0.00% <ø> (-100.00%)	⬇️
...ntend/src/explore/components/ExploreChartPanel.jsx	82.60% <ø> (ø)
superset-frontend/src/explore/reducers/index.js	100.00% <ø> (ø)
superset-frontend/src/chart/ChartRenderer.jsx	76.31% <33.33%> (+0.31%)	⬆️
superset-frontend/src/chart/chartAction.js	78.00% <50.00%> (ø)
... and 319 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 d0714a0...2d12c37. Read the comment docs.

[ktmud]

When adding a new library, it would be nice to add a note in PR description to explain why it is needed. Sometimes it's not very obvious what the purpose of the library is.