durgaprasadml opened a new pull request, #41632:
URL: https://github.com/apache/superset/pull/41632
SUMMARY
Fixes an issue where the “View as table” action fails in Embedded Dashboard
mode with the following error:
AG Grid: error #272 No AG Grid modules are registered!
The embedded dashboard application uses a separate bootstrap entrypoint from
the main Superset application. While the main application initializes AG Grid
modules during startup, the embedded bootstrap flow did not initialize them
before AG Grid components were mounted.
This change ensures AG Grid modules are initialized during embedded
application startup, aligning the embedded initialization flow with the main
application and allowing the “View as table” modal to render correctly inside
embedded dashboards.
Fixes #41630.
⸻
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before
* Clicking View as table inside an embedded dashboard opens an empty modal.
* The browser console displays:
* AG Grid: error #272 No AG Grid modules are registered!
After
* Clicking View as table correctly renders the table inside the modal.
* No AG Grid registration errors appear in the browser console.
⸻
TESTING INSTRUCTIONS
1. Enable embedded dashboards by setting:
EMBEDDED_SUPERSET = True
2. Embed a dashboard using the Embedded SDK.
3. Open the chart actions menu (...) on any chart.
4. Click View as table.
5. Verify the table renders successfully.
6. Verify no AG Grid registration errors appear in the browser console.
7. Verify the main Superset dashboard experience remains unchanged.
Additionally:
* Added regression coverage to verify AG Grid modules are initialized during
embedded application bootstrap.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
