Skip to content

[Visualize] Fix embeddable panel title behavior - #200548

Merged
dej611 merged 3 commits into
elastic:mainfrom
dej611:fix/199882
Nov 18, 2024
Merged

dej611 merged 3 commits into
elastic:mainfrom
dej611:fix/199882

Conversation

@dej611

@dej611 dej611 commented Nov 18, 2024 •

Copy link
Copy Markdown
Contributor

Summary

This PR adds a fix for a regression bug introduced with the new embeddable refactor in 8.16 .
I've added an extra 8.16 FTR test to ensure it works.

Checklist

@dej611 dej611 added Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// Feature:Embeddables Relating to the Embeddable system v8.17.0 v8.16.1 labels Nov 18, 2024
@dej611
dej611 requested a review from a team as a code owner November 18, 2024 12:18
@elasticmachine

Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-visualizations (Team:Visualizations)

@dej611 dej611 added the backport:version Backport to applied version labels label Nov 18, 2024
@dej611 dej611 added the v9.0.0 label Nov 18, 2024
@dej611
dej611 merged commit 5102b50 into elastic:main Nov 18, 2024
@kibanamachine

Copy link
Copy Markdown
Contributor

Starting backport for target branches: 8.16, 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 18, 2024
## Summary

This PR adds a fix for a regression bug introduced with the new
embeddable refactor in 8.16 .
I've added an extra 8.16 FTR test to ensure it works.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 5102b50)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 18, 2024
## Summary

This PR adds a fix for a regression bug introduced with the new
embeddable refactor in 8.16 .
I've added an extra 8.16 FTR test to ensure it works.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 5102b50)
@kibanamachine

Copy link
Copy Markdown
Contributor

💚 All backports created successfully

Status Branch Result
✅ 8.16
✅ 8.x

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 18, 2024
)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Visualize] Fix embeddable panel title behavior
(#200548)](#200548)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Marco
Liberati","email":"dej611@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-11-18T15:33:30Z","message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Visualizations","release_note:fix","Team:Visualizations","v9.0.0","Feature:Embeddables","backport:version","v8.17.0","v8.16.1"],"title":"[Visualize]
Fix embeddable panel title
behavior","number":200548,"url":"https://github.com/elastic/kibana/pull/200548","mergeCommit":{"message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653"}},"sourceBranch":"main","suggestedTargetBranches":["8.x","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200548","number":200548,"mergeCommit":{"message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Marco Liberati <dej611@users.noreply.github.com>
kibanamachine added a commit that referenced this pull request Nov 18, 2024
…0592)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Visualize] Fix embeddable panel title behavior
(#200548)](#200548)

<!--- Backport version: 9.4.3 -->

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

<!--BACKPORT [{"author":{"name":"Marco
Liberati","email":"dej611@users.noreply.github.com"},"sourceCommit":{"committedDate":"2024-11-18T15:33:30Z","message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:Visualizations","release_note:fix","Team:Visualizations","v9.0.0","Feature:Embeddables","backport:version","v8.17.0","v8.16.1"],"title":"[Visualize]
Fix embeddable panel title
behavior","number":200548,"url":"https://github.com/elastic/kibana/pull/200548","mergeCommit":{"message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653"}},"sourceBranch":"main","suggestedTargetBranches":["8.x","8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/200548","number":200548,"mergeCommit":{"message":"[Visualize]
Fix embeddable panel title behavior (#200548)\n\n## Summary\r\n\r\nThis
PR adds a fix for a regression bug introduced with the new\r\nembeddable
refactor in 8.16 .\r\nI've added an extra 8.16 FTR test to ensure it
works.\r\n\r\n\r\n### Checklist\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios","sha":"5102b504890ccdc3b6632e6fe0f39e87ee82b653"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.16","label":"v8.16.1","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Marco Liberati <dej611@users.noreply.github.com>
jesuswr pushed a commit to jesuswr/kibana that referenced this pull request Nov 18, 2024
## Summary

This PR adds a fix for a regression bug introduced with the new
embeddable refactor in 8.16 .
I've added an extra 8.16 FTR test to ensure it works.


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

This PR adds a fix for a regression bug introduced with the new
embeddable refactor in 8.16 .
I've added an extra 8.16 FTR test to ensure it works.


### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios
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 Feature:Embeddables Relating to the Embeddable system Feature:Visualizations Generic visualization features (in case no more specific feature label is available) release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v8.16.1 v8.17.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants