fix: advanced data type API spec and permission name

[dpgaspar]

SUMMARY

Simple fix for the new advanced type REST API, we currently use can_read permissions instead of FAB default can_get
also simplified the OpenAPI spec and added the rison schema to the decorator so that JSON schema will actually be evaluated

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING 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

[villebro]

LGTM, thanks for this improvement! FYI @cccs-RyanS

[codecov]

Codecov Report

Merging #20128 (7513874) into master (660af40) will decrease coverage by 0.18%.
The diff coverage is 77.77%.

❗ Current head 7513874 differs from pull request most recent head 5112dbd. Consider uploading reports for the commit 5112dbd to get more accurate results

@@            Coverage Diff             @@
##           master   #20128      +/-   ##
==========================================
- Coverage   66.47%   66.28%   -0.19%     
==========================================
  Files        1721     1721              
  Lines       64477    64481       +4     
  Branches     6795     6795              
==========================================
- Hits        42858    42743     -115     
- Misses      19891    20010     +119     
  Partials     1728     1728              

Flag	Coverage Δ
hive	?
mysql	82.14% <100.00%> (+<0.01%)	⬆️
postgres	82.20% <100.00%> (+<0.01%)	⬆️
presto	?
python	82.24% <100.00%> (-0.38%)	⬇️
sqlite	?
unit	?

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

Impacted Files	Coverage Δ
...d/packages/superset-ui-chart-controls/src/types.ts	100.00% <ø> (ø)
...egacy-plugin-chart-event-flow/src/controlPanel.tsx	14.28% <0.00%> (ø)
...chart-echarts/src/MixedTimeseries/controlPanel.tsx	92.85% <ø> (ø)
.../plugin-chart-echarts/src/MixedTimeseries/index.ts	50.00% <ø> (ø)
...hart-echarts/src/MixedTimeseries/transformProps.ts	0.00% <0.00%> (ø)
...gin-chart-echarts/src/Timeseries/transformProps.ts	56.86% <ø> (ø)
...tend/plugins/plugin-chart-echarts/src/constants.ts	100.00% <ø> (ø)
...d/plugins/plugin-chart-echarts/src/utils/series.ts	92.00% <ø> (ø)
...tend/plugins/plugin-chart-echarts/src/controls.tsx	73.91% <66.66%> (ø)
...t-frontend/src/dashboard/components/SliceAdder.jsx	69.35% <90.90%> (ø)
... and 31 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 660af40...5112dbd. Read the comment docs.

[dpgaspar]

Rison automatically parses the query string from the defined json schema

[villebro]

Good catch 👍