Skip to content

[dashboard] fix 'sync colors' and 'sync tooltips' must be ON by default#242442

Merged
nreese merged 5 commits intoelastic:mainfrom
nreese:issue_242373
Nov 11, 2025
Merged

[dashboard] fix 'sync colors' and 'sync tooltips' must be ON by default#242442
nreese merged 5 commits intoelastic:mainfrom
nreese:issue_242373

Conversation

@nreese
Copy link
Copy Markdown
Contributor

@nreese nreese commented Nov 10, 2025

Closes #242373

Regression caused by #238129. PR removed options from DEFAULT_DASHBOARD_STATE and starting using existing DEFAULT_DASHBOARD_OPTIONS. DEFAULT_DASHBOARD_OPTIONS added in #193067. The problem is that DEFAULT_DASHBOARD_OPTIONS does not have the same values as options in DEFAULT_DASHBOARD_STATE and this was missed during review.

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Nov 10, 2025

/ci

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Nov 10, 2025

/ci

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Nov 10, 2025

/ci

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Nov 10, 2025

/ci

@elasticmachine
Copy link
Copy Markdown
Contributor

⏳ Build in-progress, with failures

Failed CI Steps

Test Failures

  • [job] [logs] Jest Tests #20 / transformDashboardIn should handle missing optional state keys
  • [job] [logs] Jest Tests #20 / transformDashboardIn should handle missing optional state keys

History

@nreese
Copy link
Copy Markdown
Contributor Author

nreese commented Nov 10, 2025

/ci

@nreese nreese marked this pull request as ready for review November 10, 2025 21:20
@nreese nreese requested a review from a team as a code owner November 10, 2025 21:20
@nreese nreese added release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.3.0 backport:skip This PR does not require backporting labels Nov 10, 2025
@nickpeihl nickpeihl self-requested a review November 11, 2025 18:13
Copy link
Copy Markdown
Contributor

@nickpeihl nickpeihl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! thanks for the fix

@nreese nreese merged commit fd0545a into elastic:main Nov 11, 2025
22 checks passed
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…lt (elastic#242442)

Closes elastic#242373

Regression caused by elastic#238129. PR
removed options from `DEFAULT_DASHBOARD_STATE` and starting using
existing
[DEFAULT_DASHBOARD_OPTIONS](https://github.com/elastic/kibana/blob/main/src/platform/plugins/shared/dashboard/common/content_management/constants.ts#L19).
`DEFAULT_DASHBOARD_OPTIONS` added in
elastic#193067. The problem is that
`DEFAULT_DASHBOARD_OPTIONS` does not have the same values as options in
`DEFAULT_DASHBOARD_STATE` and this was missed during review.

---------

Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.3.0

4 participants