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] capitalize FROM in recommended queries #205122

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Conversation

drewdaemon
Copy link
Contributor

@drewdaemon drewdaemon commented Dec 23, 2024

Summary

FROM wasn't capitalized when selected from the menu:

Screen.Recording.2024-12-23.at.2.42.55.PM.mov

Now it is.

@drewdaemon drewdaemon added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Feature:ES|QL ES|QL related features in Kibana Team:ESQL ES|QL related features in Kibana backport:version Backport to applied version labels v8.18.0 labels Dec 23, 2024
@drewdaemon drewdaemon requested a review from a team as a code owner December 23, 2024 21:44
@elasticmachine
Copy link
Contributor

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

@stratoula stratoula enabled auto-merge (squash) January 2, 2025 07:06
@stratoula stratoula merged commit e5cf28b into main Jan 2, 2025
8 checks passed
@stratoula stratoula deleted the drewdaemon-patch-1 branch January 2, 2025 08:30
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

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

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 2, 2025
## Summary

`FROM` wasn't capitalized when selected from the menu:

https://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc

Now it is.

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
(cherry picked from commit e5cf28b)
@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 Jan 2, 2025
…5122) (#205352)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[ES|QL] capitalize &#x60;FROM&#x60; in recommended queries
(#205122)](#205122)

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

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

<!--BACKPORT [{"author":{"name":"Drew
Tate","email":"drew.tate@elastic.co"},"sourceCommit":{"committedDate":"2025-01-02T08:30:42Z","message":"[ES|QL]
capitalize `FROM` in recommended queries (#205122)\n\n##
Summary\r\n\r\n`FROM` wasn't capitalized when selected from the
menu:\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc\r\n\r\n\r\n\r\nNow
it is.\r\n\r\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>","sha":"e5cf28bc27b6ca80c92c44a4fc805adce857b518","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.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.18.0"],"title":"[ES|QL]
capitalize `FROM` in recommended
queries","number":205122,"url":"https://github.com/elastic/kibana/pull/205122","mergeCommit":{"message":"[ES|QL]
capitalize `FROM` in recommended queries (#205122)\n\n##
Summary\r\n\r\n`FROM` wasn't capitalized when selected from the
menu:\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc\r\n\r\n\r\n\r\nNow
it is.\r\n\r\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>","sha":"e5cf28bc27b6ca80c92c44a4fc805adce857b518"}},"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/205122","number":205122,"mergeCommit":{"message":"[ES|QL]
capitalize `FROM` in recommended queries (#205122)\n\n##
Summary\r\n\r\n`FROM` wasn't capitalized when selected from the
menu:\r\n\r\n\r\n\r\nhttps://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc\r\n\r\n\r\n\r\nNow
it is.\r\n\r\nCo-authored-by: Stratoula Kalafateli
<efstratia.kalafateli@elastic.co>","sha":"e5cf28bc27b6ca80c92c44a4fc805adce857b518"}},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

---------

Co-authored-by: Drew Tate <drew.tate@elastic.co>
benakansara pushed a commit to benakansara/kibana that referenced this pull request Jan 2, 2025
## Summary

`FROM` wasn't capitalized when selected from the menu:



https://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc



Now it is.

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
cqliu1 pushed a commit to cqliu1/kibana that referenced this pull request Jan 2, 2025
## Summary

`FROM` wasn't capitalized when selected from the menu:



https://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc



Now it is.

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Jan 13, 2025
## Summary

`FROM` wasn't capitalized when selected from the menu:



https://github.com/user-attachments/assets/939c24b6-5ae5-417b-9dfa-fb4d411b8abc



Now it is.

Co-authored-by: Stratoula Kalafateli <efstratia.kalafateli@elastic.co>
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.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants