fix: key error on permalink fetch for old permalinks

[eschutho]

SUMMARY

Adds in backward compatibility for old permalink keys that may have datasetId instead of datasourceId

TESTING INSTRUCTIONS

Testing requires an old permalink, so it will be difficult to test on ephemeral. You can do regression testing on ephemeral for new keys by creating and viewing permalinks from explore.

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

[eschutho]

Fixes a bug from #19981

[codecov]

Codecov Report

Merging #20414 (69571a1) into master (ead1040) will decrease coverage by 0.04%.
The diff coverage is 90.00%.

❗ Current head 69571a1 differs from pull request most recent head dd3b634. Consider uploading reports for the commit dd3b634 to get more accurate results

@@            Coverage Diff             @@
##           master   #20414      +/-   ##
==========================================
- Coverage   66.69%   66.65%   -0.05%     
==========================================
  Files        1739     1739              
  Lines       65153    65136      -17     
  Branches     6903     6897       -6     
==========================================
- Hits        43453    43415      -38     
- Misses      19947    19968      +21     
  Partials     1753     1753              

Flag	Coverage Δ
hive	53.70% <40.00%> (+<0.01%)	⬆️
mysql	82.29% <100.00%> (+<0.01%)	⬆️
postgres	82.36% <100.00%> (+<0.01%)	⬆️
presto	?
python	82.67% <100.00%> (-0.12%)	⬇️
sqlite	82.09% <100.00%> (+<0.01%)	⬆️
unit	?

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

Impacted Files	Coverage Δ
...art-controls/src/operators/contributionOperator.ts	100.00% <ø> (ø)
...ui-chart-controls/src/operators/flattenOperator.ts	100.00% <ø> (ø)
...ui-chart-controls/src/operators/prophetOperator.ts	100.00% <ø> (ø)
...i-chart-controls/src/operators/resampleOperator.ts	100.00% <ø> (ø)
...chart-echarts/src/MixedTimeseries/controlPanel.tsx	65.00% <ø> (ø)
...gins/plugin-chart-echarts/src/Pie/controlPanel.tsx	28.57% <0.00%> (-4.77%)	⬇️
...chart-echarts/src/Timeseries/Area/controlPanel.tsx	33.33% <ø> (ø)
...charts/src/Timeseries/Regular/Bar/controlPanel.tsx	33.33% <ø> (ø)
...ts/src/Timeseries/Regular/Scatter/controlPanel.tsx	33.33% <ø> (ø)
...rt-echarts/src/Timeseries/Regular/controlPanel.tsx	33.33% <ø> (ø)
... and 29 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 ead1040...dd3b634. Read the comment docs.

[betodealmeida]

Looks good!