Skip to content

[links] fix Can not save Dashboard after switching a Dashboard Link to an External Link#243134

Merged
nreese merged 8 commits intoelastic:mainfrom
nreese:issue_242969
Nov 17, 2025
Merged

[links] fix Can not save Dashboard after switching a Dashboard Link to an External Link#243134
nreese merged 8 commits intoelastic:mainfrom
nreese:issue_242969

Conversation

@nreese
Copy link
Contributor

@nreese nreese commented Nov 16, 2025

Closes #242969

Test instructions

  • install web logs sample data
  • create new dashboard
  • add new links panel
  • add new dashboard link
  • Change dashboard link option
  • before clicking "Add link", switch to external link
  • set url
  • click add link
  • click saved links panel
  • save dashboard. Dashboard should save without any errors.
@nreese
Copy link
Contributor Author

nreese commented Nov 16, 2025

/ci

@nreese
Copy link
Contributor Author

nreese commented Nov 17, 2025

@elasticmachine merge upstream

@nreese
Copy link
Contributor Author

nreese commented Nov 17, 2025

/ci

@nreese
Copy link
Contributor Author

nreese commented Nov 17, 2025

/ci

@nreese
Copy link
Contributor Author

nreese commented Nov 17, 2025

@elasticmachine merge upstream

@nreese
Copy link
Contributor Author

nreese commented Nov 17, 2025

/ci

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
links 138 139 +1

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
links 98.6KB 99.1KB +451.0B

History

@nreese nreese marked this pull request as ready for review November 17, 2025 20:58
@nreese nreese requested a review from a team as a code owner November 17, 2025 20:58
@nreese nreese added release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// backport:version Backport to applied version labels v9.3.0 v9.2.2 labels Nov 17, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

Copy link
Contributor

@ThomThomson ThomThomson left a comment

Choose a reason for hiding this comment

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

Changes LGTM - well tested and written! Code only review.

@nreese nreese merged commit 5fece3e into elastic:main Nov 17, 2025
24 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 9.2

https://github.com/elastic/kibana/actions/runs/19446439210

kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Nov 17, 2025
…o an External Link (elastic#243134)

Closes elastic#242969

### Test instructions
* install web logs sample data
* create new dashboard
* add new links panel
* add new dashboard link
* Change dashboard link option
* before clicking "Add link", switch to external link
* set url
* click add link
* click saved links panel
* save dashboard. Dashboard should save without any errors.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 5fece3e)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
9.2

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 17, 2025
…Link to an External Link (#243134) (#243295)

# Backport

This will backport the following commits from `main` to `9.2`:
- [[links] fix Can not save Dashboard after switching a Dashboard Link
to an External Link
(#243134)](#243134)

<!--- Backport version: 9.6.6 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sorenlouv/backport)

<!--BACKPORT [{"author":{"name":"Nathan
Reese","email":"reese.nathan@elastic.co"},"sourceCommit":{"committedDate":"2025-11-17T22:17:03Z","message":"[links]
fix Can not save Dashboard after switching a Dashboard Link to an
External Link (#243134)\n\nCloses
https://github.com/elastic/kibana/issues/242969\n\n### Test
instructions\n* install web logs sample data\n* create new dashboard\n*
add new links panel\n* add new dashboard link\n* Change dashboard link
option\n* before clicking \"Add link\", switch to external link\n* set
url\n* click add link\n* click saved links panel\n* save dashboard.
Dashboard should save without any
errors.\n\n---------\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5fece3ed46bac79d6482dc575a74a42d9397cc85","branchLabelMapping":{"^v9.3.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","Team:Presentation","backport:version","v9.3.0","v9.2.2"],"title":"[links]
fix Can not save Dashboard after switching a Dashboard Link to an
External
Link","number":243134,"url":"https://github.com/elastic/kibana/pull/243134","mergeCommit":{"message":"[links]
fix Can not save Dashboard after switching a Dashboard Link to an
External Link (#243134)\n\nCloses
https://github.com/elastic/kibana/issues/242969\n\n### Test
instructions\n* install web logs sample data\n* create new dashboard\n*
add new links panel\n* add new dashboard link\n* Change dashboard link
option\n* before clicking \"Add link\", switch to external link\n* set
url\n* click add link\n* click saved links panel\n* save dashboard.
Dashboard should save without any
errors.\n\n---------\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5fece3ed46bac79d6482dc575a74a42d9397cc85"}},"sourceBranch":"main","suggestedTargetBranches":["9.2"],"targetPullRequestStates":[{"branch":"main","label":"v9.3.0","branchLabelMappingKey":"^v9.3.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/243134","number":243134,"mergeCommit":{"message":"[links]
fix Can not save Dashboard after switching a Dashboard Link to an
External Link (#243134)\n\nCloses
https://github.com/elastic/kibana/issues/242969\n\n### Test
instructions\n* install web logs sample data\n* create new dashboard\n*
add new links panel\n* add new dashboard link\n* Change dashboard link
option\n* before clicking \"Add link\", switch to external link\n* set
url\n* click add link\n* click saved links panel\n* save dashboard.
Dashboard should save without any
errors.\n\n---------\n\nCo-authored-by: Elastic Machine
<elasticmachine@users.noreply.github.com>\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"5fece3ed46bac79d6482dc575a74a42d9397cc85"}},{"branch":"9.2","label":"v9.2.2","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Nathan Reese <reese.nathan@elastic.co>
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
…o an External Link (elastic#243134)

Closes elastic#242969

### Test instructions
* install web logs sample data
* create new dashboard
* add new links panel
* add new dashboard link
* Change dashboard link option
* before clicking "Add link", switch to external link
* set url
* click add link
* click saved links panel
* save dashboard. Dashboard should save without any errors.

---------

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
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:version Backport to applied version labels release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.2.2 v9.3.0

4 participants