Skip to content

Silence URL restore errors for Discover & Dashboards#242788

Merged
ThomThomson merged 6 commits intoelastic:mainfrom
ThomThomson:urls/silenceRestoreError
Nov 24, 2025
Merged

Silence URL restore errors for Discover & Dashboards#242788
ThomThomson merged 6 commits intoelastic:mainfrom
ThomThomson:urls/silenceRestoreError

Conversation

@ThomThomson
Copy link
Contributor

@ThomThomson ThomThomson commented Nov 12, 2025

Summary

Fixes #242783 by silencing the error notifications for Discover and Dashboards, and by changing the built in URL restore error to a console.warn.

As a more drastic followup, the setting is deprecated, and the URL length limit code that recommends it is removed in #242972

@ThomThomson
Copy link
Contributor Author

/ci

@ThomThomson
Copy link
Contributor Author

/ci

@ThomThomson
Copy link
Contributor Author

@elasticmachine merge upstream

@ThomThomson
Copy link
Contributor Author

/ci

@ThomThomson
Copy link
Contributor Author

@elasticmachine merge upstream

@ThomThomson
Copy link
Contributor Author

/ci

@ThomThomson ThomThomson added Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// backport:skip This PR does not require backporting loe:small Small Level of Effort impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. labels Nov 20, 2025
@ThomThomson ThomThomson marked this pull request as ready for review November 20, 2025 20:41
@ThomThomson ThomThomson requested review from a team as code owners November 20, 2025 20:41
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

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

kibana-presentation changes LGTM
code review only

Copy link
Contributor

@AlexGPlay AlexGPlay left a comment

Choose a reason for hiding this comment

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

data discovery changes LGTM! 🚀

@ThomThomson
Copy link
Contributor Author

@elasticmachine merge upstream

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

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

id before after diff
dashboard 673.6KB 673.5KB -110.0B
discover 1.2MB 1.2MB -86.0B
total -196.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
kibanaUtils 48.7KB 48.7KB +11.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
kibanaUtils 15 16 +1

Total ESLint disabled count

id before after diff
kibanaUtils 17 18 +1

History

@ThomThomson ThomThomson merged commit f4d0e3d into elastic:main Nov 24, 2025
12 checks passed
eokoneyo pushed a commit to eokoneyo/kibana that referenced this pull request Dec 2, 2025
Silences the annoying URL restoration error notifications for Discover and Dashboards
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 impact:medium Addressing this issue will have a medium level of impact on the quality/strength of our product. loe:small Small Level of Effort release_note:fix Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas t// v9.3.0

6 participants