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

[ES|QL] Fixes the wrongly focused suggestion #198654

Merged
merged 4 commits into from
Nov 4, 2024

Conversation

stratoula
Copy link
Contributor

@stratoula stratoula commented Nov 1, 2024

Summary

Removes the wrongly focused panel id.

Now

image

Before
image

@stratoula stratoula added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 backport:version Backport to applied version labels v8.17.0 Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana labels Nov 1, 2024
@stratoula stratoula marked this pull request as ready for review November 1, 2024 11:05
@stratoula stratoula requested a review from a team as a code owner November 1, 2024 11:05
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-esql (Team:ESQL)

@stratoula stratoula enabled auto-merge (squash) November 4, 2024 12:39
@stratoula stratoula merged commit 867b05e into elastic:main Nov 4, 2024
23 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

@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
unifiedSearch 353.1KB 353.1KB -26.0B

History

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

Removes the wrongly focused panel id.

**Now**

<img width="421" alt="image"
src="https://github.com/user-attachments/assets/31ab675a-349a-49ca-a2aa-8272ff88319d">

**Before**
<img width="524" alt="image"
src="https://github.com/user-attachments/assets/0483f46c-3cfb-48b7-b32e-ff15e2a285ff">

(cherry picked from commit 867b05e)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
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 4, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] Fixes the wrongly focused suggestion
(#198654)](#198654)

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

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

<!--BACKPORT [{"author":{"name":"Stratoula
Kalafateli","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-04T13:27:43Z","message":"[ES|QL]
Fixes the wrongly focused suggestion (#198654)\n\n##
Summary\r\n\r\nRemoves the wrongly focused panel
id.\r\n\r\n**Now**\r\n\r\n<img width=\"421\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/31ab675a-349a-49ca-a2aa-8272ff88319d\">\r\n\r\n**Before**\r\n<img
width=\"524\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/0483f46c-3cfb-48b7-b32e-ff15e2a285ff\">","sha":"867b05eb586d2d9f36547b4d6dc2a20d975de7a8","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Feature:ES|QL","Team:ESQL","backport:version","v8.17.0"],"title":"[ES|QL]
Fixes the wrongly focused
suggestion","number":198654,"url":"https://github.com/elastic/kibana/pull/198654","mergeCommit":{"message":"[ES|QL]
Fixes the wrongly focused suggestion (#198654)\n\n##
Summary\r\n\r\nRemoves the wrongly focused panel
id.\r\n\r\n**Now**\r\n\r\n<img width=\"421\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/31ab675a-349a-49ca-a2aa-8272ff88319d\">\r\n\r\n**Before**\r\n<img
width=\"524\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/0483f46c-3cfb-48b7-b32e-ff15e2a285ff\">","sha":"867b05eb586d2d9f36547b4d6dc2a20d975de7a8"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198654","number":198654,"mergeCommit":{"message":"[ES|QL]
Fixes the wrongly focused suggestion (#198654)\n\n##
Summary\r\n\r\nRemoves the wrongly focused panel
id.\r\n\r\n**Now**\r\n\r\n<img width=\"421\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/31ab675a-349a-49ca-a2aa-8272ff88319d\">\r\n\r\n**Before**\r\n<img
width=\"524\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/0483f46c-3cfb-48b7-b32e-ff15e2a285ff\">","sha":"867b05eb586d2d9f36547b4d6dc2a20d975de7a8"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Stratoula Kalafateli <[email protected]>
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:ES|QL ES|QL related features in Kibana release_note:skip Skip the PR/issue when compiling release notes Team:ESQL ES|QL related features in Kibana v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants