Skip to content

Conversation

@DiogoVala
Copy link
Contributor

Functionality allows external scripts to be specified in themes and loaded at runtime.
Useful to embed functionality without modifying datahub code. In our case we are using it for a feedback widget which sits on top of Datahub and allows users to suggest improvements or point out issues.

@github-actions github-actions bot added product PR or Issue related to the DataHub UI/UX devops PR or Issue related to DataHub backend & deployment community-contribution PR or Issue raised by member(s) of DataHub Community labels Dec 17, 2025
@DiogoVala DiogoVala changed the title Allow external scripts in custom themes feat(frontend): Allow external scripts in custom themes Dec 17, 2025
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label Dec 17, 2025
@codecov
Copy link

codecov bot commented Dec 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Dec 17, 2025

Bundle Report

Changes will increase total bundle size by 241 bytes (0.0%) ⬆️. This is within the configured threshold ✅

Detailed changes
Bundle name Size Change
datahub-react-web-esm 31.27MB 241 bytes (0.0%) ⬆️

Affected Assets, Files, and Routes:

view changes for bundle: datahub-react-web-esm

Assets Changed:

Asset Name Size Change Total Size Change (%)
assets/index-*.js 241 bytes 19.24MB 0.0%

Files in assets/index-*.js:

  • ./src/app/ExternalScriptLoader.tsx → Total Size: 460 bytes

  • ./src/App.tsx → Total Size: 2.35kB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution PR or Issue raised by member(s) of DataHub Community devops PR or Issue related to DataHub backend & deployment needs-review Label for PRs that need review from a maintainer. product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants