fix: propagate color mapping from dashboard to charts

[khtruong]

CATEGORY

Choose one

• Bug Fix
• Enhancement (new features, refinement)
• Refactor
• Add tests
• Build / Development Environment
• Documentation

SUMMARY

If a dashboard has a color mapping saved and the user explores a chart from the dashboard, the coloring may look different. This is because we would propagate the color scheme but not the color mapping to the chart. This change fixes this issue. The PR also sets it up for color consistency in charts and the ability to change an individual color mapping in the future.

TEST PLAN

Tested manually

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

REVIEWERS

@graceguo-supercat @kristw @williaster @betodealmeida @xtinec @DiggidyDave

[codecov-io]

Codecov Report

Merging #7289 into lyftga will decrease coverage by 0.01%.
The diff coverage is 46.66%.

@@            Coverage Diff             @@
##           lyftga    #7289      +/-   ##
==========================================
- Coverage   64.58%   64.57%   -0.02%     
==========================================
  Files         425      426       +1     
  Lines       20884    20898      +14     
  Branches     2297     2297              
==========================================
+ Hits        13488    13494       +6     
- Misses       7270     7278       +8     
  Partials      126      126

Impacted Files	Coverage Δ
...et/assets/src/explore/components/controls/index.js	100% <ø> (ø)	⬆️
...et/assets/src/explore/controlPanels/DeckScatter.js	0% <ø> (ø)	⬆️
...rset/assets/src/explore/controlPanels/sections.jsx	100% <ø> (ø)	⬆️
superset/assets/src/explore/controls.jsx	42.06% <0%> (-0.34%)	⬇️
...shboard/util/charts/getFormDataWithExtraFilters.js	90% <100%> (+1.11%)	⬆️
superset/assets/src/dashboard/containers/Chart.jsx	90.9% <100%> (ø)	⬆️
...rc/explore/components/controls/ColorMapControl.jsx	36.36% <36.36%> (ø)

---

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 65cc0e5...85961e4. Read the comment docs.

[codecov-io]

Codecov Report

Merging #7289 into lyftga will decrease coverage by 0.01%.
The diff coverage is 52.94%.

@@            Coverage Diff             @@
##           lyftga    #7289      +/-   ##
==========================================
- Coverage   64.58%   64.57%   -0.02%     
==========================================
  Files         425      426       +1     
  Lines       20884    20900      +16     
  Branches     2297     2299       +2     
==========================================
+ Hits        13488    13496       +8     
- Misses       7270     7278       +8     
  Partials      126      126

Impacted Files	Coverage Δ
...et/assets/src/explore/components/controls/index.js	100% <ø> (ø)	⬆️
...et/assets/src/explore/controlPanels/DeckScatter.js	0% <ø> (ø)	⬆️
...rset/assets/src/explore/controlPanels/sections.jsx	100% <ø> (ø)	⬆️
...rset/assets/src/dashboard/components/Dashboard.jsx	89.28% <ø> (ø)	⬆️
superset/assets/src/explore/controls.jsx	42.06% <0%> (-0.34%)	⬇️
...shboard/util/charts/getFormDataWithExtraFilters.js	90.9% <100%> (+2.02%)	⬆️
superset/assets/src/dashboard/containers/Chart.jsx	90.9% <100%> (ø)	⬆️
...rc/explore/components/controls/ColorMapControl.jsx	36.36% <36.36%> (ø)
...re/components/controls/TimeSeriesColumnControl.jsx	65.9% <0%> (ø)	⬆️

---

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 65cc0e5...5377c82. Read the comment docs.

[graceguo-supercat]

LGTM

[lilila]

Dear @khtruong , do you know why color mapping fails when charts are rendered in Standalone mode? Would you mind helping fixing this issue?