Skip to content

feat: DH-21757: Allow widget plugins to register types as dashboard types#2679

Open
mofojed wants to merge 1 commit into
deephaven:mainfrom
mofojed:DH-21757-widget-plugin-dashboard-types
Open

feat: DH-21757: Allow widget plugins to register types as dashboard types#2679
mofojed wants to merge 1 commit into
deephaven:mainfrom
mofojed:DH-21757-widget-plugin-dashboard-types

Conversation

@mofojed
Copy link
Copy Markdown
Member

@mofojed mofojed commented May 13, 2026

  • Allows widget plugins to register a type as a dashboard type, and create the dashboard payload
    • Avoids the current hack where the ui DashboardPlugin intercepts the panel open and then emits a dashboard creation, is more explicit about how it's handled
    • Also allows other plugins to potential register dashboard types
  • Used by feat: DH-21757: WidgetPlugin support in deephaven.ui deephaven-plugins#1341, Enterprise draft at https://github.com/deephaven-ent/iris/pull/4653
  • Leaves the door open for if we want to have an option for allowing dashboards to open as a panel by default (for now, you can just wrap a dashboard in a ui.panel)

@mofojed mofojed requested a review from dgodinez-dh May 13, 2026 21:11
@mofojed mofojed self-assigned this May 13, 2026
…ypes

- Allows widget plugins to register a type as a dashboard type, and create the dashboard payload
  - Avoids the current hack where the ui DashboardPlugin intercepts the panel open and then emits a dashboard creation, is more explicit about how it's handled
  - Also allows other plugins to potential register dashboard types
@mofojed mofojed force-pushed the DH-21757-widget-plugin-dashboard-types branch from fdfefce to eac1baa Compare May 13, 2026 21:22
@mofojed mofojed marked this pull request as ready for review May 13, 2026 21:22
@mofojed mofojed requested a review from vbabich May 13, 2026 21:22
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.

1 participant