Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.17] [Dashboard] Fix blank panel save and display issue. (#120815) #121661

Closed
wants to merge 1 commit into from

Conversation

Heenawter
Copy link
Contributor

Backports the following commits to 7.17:

* Added special case to panel comparison.

* Fixed panel title behaviour when linking/unlinking to/from the library.

* Added panel title test suite.

* Made the injection of the 'savedSearchId' key conditional on if it is defined when unlinking.

* Adjusted use of 'setValue()' based on feedback from @dmlemeshko

* Refractor test suite.

* Improved code for removal of title on library link.
# Conflicts:
#	src/plugins/embeddable/public/lib/attribute_service/attribute_service.tsx
@Heenawter Heenawter enabled auto-merge (squash) December 20, 2021 16:30
@kibana-ci
Copy link
Collaborator

kibana-ci commented Dec 20, 2021

💔 Build Failed

Failed CI Steps

Test Failures

  • [job] [logs] Default CI Group #19 / dashboard panel titles "before all" hook in "panel titles"
  • [job] [logs] Default CI Group #19 / dashboard panel titles "before all" hook in "panel titles"
  • [job] [logs] Default CI Group #19 / dashboard panel titles "before all" hook in "panel titles"

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 141.4KB 141.6KB +233.0B
visualizations 73.0KB 73.1KB +32.0B
total +265.0B

Page load bundle

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

id before after diff
dashboard 167.9KB 168.0KB +65.0B
embeddable 68.7KB 68.8KB +130.0B
total +195.0B

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@Heenawter Heenawter closed this Dec 20, 2021
auto-merge was automatically disabled December 20, 2021 20:21

Pull request was closed

@Heenawter Heenawter deleted the backport/7.17/pr-120815 branch December 20, 2021 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants