fix: Right-click on a chart's null values

[michael-s-molina]

SUMMARY

Fixes a bug when right-clicking on a chart's null values.

@codyml

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Screen.Recording.2022-10-05.at.2.23.52.PM.mov

Screen.Recording.2022-10-05.at.2.19.51.PM.mov

TESTING INSTRUCTIONS

Check the videos for instructions.

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• 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

[codecov]

Codecov Report

Merging #21706 (b5aac09) into master (2907869) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #21706      +/-   ##
==========================================
- Coverage   66.84%   66.83%   -0.01%     
==========================================
  Files        1799     1799              
  Lines       68870    68889      +19     
  Branches     7314     7328      +14     
==========================================
+ Hits        46034    46043       +9     
- Misses      20954    20959       +5     
- Partials     1882     1887       +5     

Flag	Coverage Δ
javascript	53.13% <0.00%> (-0.01%)	⬇️

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

Impacted Files	Coverage Δ
...rts/src/MixedTimeseries/EchartsMixedTimeseries.tsx	0.00% <0.00%> (ø)
...chart-echarts/src/Timeseries/EchartsTimeseries.tsx	0.00% <0.00%> (ø)
...lugin-chart-echarts/src/Treemap/EchartsTreemap.tsx	0.00% <0.00%> (ø)
...ns/plugin-chart-echarts/src/utils/eventHandlers.ts	0.00% <0.00%> (ø)
...s/plugin-chart-pivot-table/src/PivotTableChart.tsx	0.00% <0.00%> (ø)
...tend/plugins/plugin-chart-table/src/TableChart.tsx	45.94% <0.00%> (-0.51%)	⬇️
.../src/dashboard/components/DrillDetailPane/utils.ts	57.14% <ø> (ø)
superset-frontend/src/preamble.ts	44.00% <0.00%> (-2.16%)	⬇️
...ntend/src/explore/components/ExploreChartPanel.jsx	69.73% <0.00%> (-1.89%)	⬇️
...frontend/src/components/DatabaseSelector/index.tsx	92.42% <0.00%> (-1.52%)	⬇️
... and 10 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[zhaoyongjie]

Hi @michael-s-molina, Thanks for the fix. from the error logging, I think it was a backend-side validation error. the Marshmallow validation might need to support null value validation. I will check this logic today.

[zhaoyongjie]

Open a PR to fix this.

[michael-s-molina]

Closing this in favor of #21713.