Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add embedded properties to Dashboard [DHIS2-18239] #18843

Closed
wants to merge 38 commits into from

Conversation

larshelge
Copy link
Member

@larshelge larshelge commented Oct 16, 2024

This PR adds a object EmbeddedDashboard properties to the Dashboard entity meant to represent an embedded dashboard.

  • provider refers to the provider of embedded dashboards, and can be extended to Power BI, Tableau and the like later.
  • id refers to the identifier of the embedded dashboard defined by the external provider.
  • options refers to customization options from the embedded dashboard provider, and can be extended later.

Adds a system setting for enabling Superset embedded dashboards.

@larshelge larshelge changed the title [WIP] feat: Add embedded dashboard properties to Dashboard [WIP] feat: Add embedded dashboard properties to Dashboard [DHIS2-18239] Oct 16, 2024
@larshelge larshelge changed the title [WIP] feat: Add embedded dashboard properties to Dashboard [DHIS2-18239] [WIP] feat: Add embedded properties to Dashboard [DHIS2-18239] Oct 16, 2024
@larshelge larshelge changed the title [WIP] feat: Add embedded properties to Dashboard [DHIS2-18239] feat: Add embedded properties to Dashboard [DHIS2-18239] Oct 16, 2024
@larshelge larshelge marked this pull request as ready for review October 23, 2024 11:08
@larshelge larshelge enabled auto-merge (squash) October 24, 2024 20:49
Copy link

sonarcloud bot commented Nov 1, 2024

@larshelge
Copy link
Member Author

I moved this PR to a new one due to the signed commit requirement in master.

#19016

@larshelge
Copy link
Member Author

Merged the new PR now. Closing this PR.

#19016

@larshelge larshelge closed this Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants