fix: dist bar will render value multi times

[Mikedu1988]

🐛 Bug Fix
related issue: #712

Now irrelevantt records will be removed, only the last applied breakpoints will be rendered.

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/superset/superset-ui/92rbm30wb
✅ Preview: https://superset-ui-git-fork-mikedu1988-fix-distbar.superset.vercel.app

[codecov]

Codecov Report

Merging #713 into master will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #713      +/-   ##
==========================================
- Coverage   24.19%   24.19%   -0.01%     
==========================================
  Files         339      339              
  Lines        7617     7618       +1     
  Branches      925      925              
==========================================
  Hits         1843     1843              
- Misses       5701     5702       +1     
  Partials       73       73              

Impacted Files	Coverage Δ
plugins/legacy-preset-chart-nvd3/src/utils.js	14.18% <0.00%> (-0.11%)	⬇️

---

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 23437fa...3fe1fcf. Read the comment docs.

[Mikedu1988]

@ktmud

[ktmud]

Thanks for the fix. I'll try to reproduce and verify later this week.

[ktmud]

Was able to reproduce in https://superset-ui.superset.vercel.app/?path=/story/legacy-chart-plugins-legacy-preset-chart-nvd3-bar--stacked-bar-with-values

and verified it works in the branch: https://superset-ui-git-fork-mikedu1988-fix-distbar.superset.vercel.app/?path=/story/legacy-chart-plugins-legacy-preset-chart-nvd3-bar--stacked-bar-with-values

[Mikedu1988]

juse one quick question, we forked and built our own superset docker image, how can we pick up this bug fix? should we just try a rebuild? or we need to bump the version for the plugin here and then we build our image based on the newer version. @ktmud

[villebro]

@Mikedu1988 this is now released as version 0.14.21 of @superset-ui/legacy-preset-chart-nvd3 (will be bumped in main Superset repo shortly)