Skip to content

Comments

[8.19] [Incident Management] [Related dashboards api] Handle missing lens attributes gracefully (#223619)#224545

Merged
dominiqueclarke merged 1 commit intoelastic:8.19from
dominiqueclarke:backport/8.19/pr-223619
Jun 19, 2025
Merged

[8.19] [Incident Management] [Related dashboards api] Handle missing lens attributes gracefully (#223619)#224545
dominiqueclarke merged 1 commit intoelastic:8.19from
dominiqueclarke:backport/8.19/pr-223619

Conversation

@dominiqueclarke
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

…tributes gracefully (elastic#223619)

## Summary

Relates to elastic#212125

When a Lens panel is saved to a dashboards, it is either saved by value
or by reference. If it's saved by reference, the attributes will not be
available. This was creating a reference error when attempting to
inspect the lens configuration to make suggestions.

This PR confirms that the attributes are available.

A follow up PR will fetch the values for the references to include them
in the evaluation.

(cherry picked from commit f6da3f2)
@dominiqueclarke dominiqueclarke added the backport This PR is a backport of another PR label Jun 19, 2025
@dominiqueclarke dominiqueclarke enabled auto-merge (squash) June 19, 2025 10:35
@botelastic botelastic bot added the Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics. label Jun 19, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-management-team (Team:obs-ux-management)

@cesco-f cesco-f self-requested a review June 19, 2025 10:39
@dominiqueclarke dominiqueclarke merged commit 0d73c32 into elastic:8.19 Jun 19, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR Team:actionable-obs Formerly "obs-ux-management", responsible for SLO, o11y alerting, significant events, & synthetics.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants