Skip to content

Conversation

@DominikB2014
Copy link
Contributor

Prevents the "detail" is not a valid choice error when trying to save a details widget
complements #104059

@DominikB2014 DominikB2014 requested a review from a team November 26, 2025 19:16
@linear
Copy link

linear bot commented Nov 26, 2025

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 26, 2025
@DominikB2014 DominikB2014 merged commit c871df2 into master Nov 27, 2025
66 of 67 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/browse-145-register-details-widget-as-supported-on-the-backend branch November 27, 2025 15:22
DominikB2014 added a commit that referenced this pull request Nov 27, 2025
1. Adds a detail widget. This widget accepts a query and will render the
first example that matches the query based on the type of span it is.
<img width="1090" height="859" alt="image"
src="https://github.com/user-attachments/assets/fb97bdd2-9c3e-4d6f-a670-23b4d8cc4692"
/>

- For now, this only applies to spans and i'm just using the
existing`fullSpanDescription` component to do this, but we can iterate
on this later on.
- This is conditionally added as an option in the widget type dropdown
if the feature flag is on.
- There's still some UI jank to sort out, i figured it's easier to split
it up so the PR doesn't get too large.
- I still need to register this on the backend, this will allow me to
save a widget of type `details` and test it properly
(#104062)

2. Adds a `isChartDisplayType` function. I noticed there's a TON of
places that checks isChartWidget. Now that we have 3 non-chart displays
i just consolidated the logic to one place.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants