fix: Pie chart not displayed in viz type selection

[zhaoyongjie]

SUMMARY

fix can not be displayed chart when some chart support cross filter behavior.

closes: #13962

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

before

after

TEST PLAN

tested in my local. would be adding more UT/IT for this case in the future.

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

[villebro]

Thanks for fixing!

[codecov]

Codecov Report

Merging #13987 (3e6eb25) into master (ab0f5e3) will increase coverage by 0.58%.
The diff coverage is 85.29%.

❗ Current head 3e6eb25 differs from pull request most recent head bfbc96b. Consider uploading reports for the commit bfbc96b to get more accurate results

@@            Coverage Diff             @@
##           master   #13987      +/-   ##
==========================================
+ Coverage   78.54%   79.13%   +0.58%     
==========================================
  Files         935      936       +1     
  Lines       47374    47409      +35     
  Branches     5956     5943      -13     
==========================================
+ Hits        37211    37515     +304     
+ Misses      10020     9767     -253     
+ Partials      143      127      -16     

Flag	Coverage Δ
cypress	56.04% <75.00%> (+<0.01%)	⬆️
javascript	68.45% <76.47%> (+1.15%)	⬆️
presto	?

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

Impacted Files	Coverage Δ
...rset-frontend/src/components/ErrorMessage/types.ts	100.00% <ø> (ø)
...shboard/components/filterscope/FilterFieldItem.jsx	100.00% <ø> (+50.00%)	⬆️
...shboard/components/filterscope/FilterFieldTree.jsx	66.66% <ø> (+22.22%)	⬆️
...hboard/components/filterscope/FilterScopeModal.tsx	75.00% <ø> (ø)
...ard/components/filterscope/FilterScopeSelector.jsx	87.67% <ø> (+85.61%)	⬆️
...shboard/components/filterscope/FilterScopeTree.jsx	70.00% <ø> (+20.00%)	⬆️
...ponents/filterscope/renderFilterFieldTreeNodes.jsx	90.00% <ø> (+90.00%)	⬆️
...ponents/filterscope/renderFilterScopeTreeNodes.jsx	86.66% <ø> (+86.66%)	⬆️
...oard/components/nativeFilters/FilterBar/Header.tsx	87.09% <0.00%> (ø)
...uperset-frontend/src/explore/exploreUtils/index.js	71.06% <ø> (ø)
... and 33 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 ab0f5e3...bfbc96b. Read the comment docs.

[junlincc]

LGTM merge!👍

[Hemant0601]

@villebro Does this include helm chart fix too?

[junlincc]

@villebro Does this include helm chart fix too?

don't think so. what is helm chat btw? @Hemant0601

[zhaoyongjie]

@villebro Does this include helm chart fix too?

Hi, @Hemant0601
what is helm chart? or deploy Superset with helm?

[Hemant0601]

@zhaoyongjie Yes installtion of superset in kubernetes using helm chart here.

https://github.com/apache/superset/tree/master/helm/superset

[villebro]

@Hemant0601 yes this PR will also fix the problem on helm.