fix(viz-gallery): respect denylist in viz gallery

[villebro]

SUMMARY

The config parameter VIZ_TYPE_DENYLIST should make it possible to omit certain viz plugins:

superset/superset/config.py

Lines 777 to 783 in 159dcd7

	# ---------------------------------------------------
	# List of viz_types not allowed in your environment
	# For example: Disable pivot table and treemap:
	# VIZ_TYPE_DENYLIST = ['pivot_table', 'treemap']
	# ---------------------------------------------------
	VIZ_TYPE_DENYLIST: List[str] = []

However, currently the viz gallery displays all chart types from MainPresets.js despite adding specific charts to the denylist.

To test, add the following to your `superset_config.py:

VIZ_TYPE_DENYLIST = ["big_number", "big_number_total", "pie"]

AFTER

With the changes, the big number and pie charts are all removed:

BEFORE

Currently they are all displayed:

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

[codecov]

Codecov Report

Merging #22658 (aff60f1) into master (399f6e3) will decrease coverage by 0.76%.
The diff coverage is 47.36%.

❗ Current head aff60f1 differs from pull request most recent head 33a7390. Consider uploading reports for the commit 33a7390 to get more accurate results

@@            Coverage Diff             @@
##           master   #22658      +/-   ##
==========================================
- Coverage   66.35%   65.59%   -0.77%     
==========================================
  Files        1869     1869              
  Lines       71582    71582              
  Branches     7814     7814              
==========================================
- Hits        47497    46952     -545     
- Misses      22044    22589     +545     
  Partials     2041     2041              

Flag	Coverage Δ
hive	?
postgres	78.08% <ø> (?)
presto	?
python	78.15% <ø> (-1.59%)	⬇️
sqlite	76.48% <ø> (ø)
unit	?

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

Impacted Files	Coverage Δ
superset-frontend/src/SqlLab/App.jsx	0.00% <0.00%> (ø)
...et-frontend/src/dashboard/components/Dashboard.jsx	64.77% <0.00%> (ø)
superset-frontend/src/embedded/api.tsx	0.00% <ø> (ø)
superset-frontend/src/embedded/index.tsx	0.00% <ø> (ø)
...ponents/controls/VizTypeControl/VizTypeGallery.tsx	88.17% <ø> (ø)
superset-frontend/src/profile/App.tsx	0.00% <0.00%> (ø)
superset-frontend/src/profile/components/App.tsx	50.00% <ø> (ø)
...frontend/src/profile/components/CreatedContent.tsx	81.81% <ø> (ø)
...rset-frontend/src/profile/components/Favorites.tsx	77.77% <ø> (ø)
...frontend/src/profile/components/RecentActivity.tsx	50.00% <ø> (ø)
... and 141 more

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

[kgabryje]

Do we want to remove that log?

[villebro]

Well, the weird thing is, I don't really see any situation where we would ever hit a page that doesn't have bootstrap data on it. This is related to GAQ, and GAQ is definitely always called from some place where there's bootstrap data. So I think it should be safe to remove (incidentally, I'll probably be working on GAQ soon, so I can address this later if it's really needed).

[kgabryje]

1 tiny nit, LGTM