[ES|QL] Suggest adding curly braces after the WITH keyword for Rerank and Completion#243047
[ES|QL] Suggest adding curly braces after the WITH keyword for Rerank and Completion#243047bartoval merged 6 commits intoelastic:mainfrom
Conversation
src/platform/packages/shared/kbn-esql-ast/src/commands_registry/complete_items.ts
Outdated
Show resolved
Hide resolved
3f0f4b7 to
16e5f02
Compare
|
Pinging @elastic/kibana-esql (Team:ESQL) |
💚 Build Succeeded
Metrics [docs]Public APIs missing comments
Page load bundle
History
cc @bartoval |
|
From the comment on the issue it seems to me that the idea is to have a suggestion after the space. Maybe the problem is just the choice of label for the suggestion? We can change it to inference_id. We can also insert all { inferende_id: ""} in one go. I had done it but it seemed like a different behavior from the current workflow |
That one! Yes I tried it!
I know that the description is confusing but what we need is to insert it directly as we do with the other flow |
09d1e2d to
24307c9
Compare
stratoula
left a comment
There was a problem hiding this comment.
This looks great, just a small change on the icon but I am approving as I dont need to check again
src/platform/packages/shared/kbn-esql-ast/src/commands_registry/complete_items.ts
Outdated
Show resolved
Hide resolved
…y/complete_items.ts Co-authored-by: Stratou <stratoula1@gmail.com>
… and Completion (elastic#243047) ## Summary elastic#234488 <img width="857" height="134" alt="iii" src="https://github.com/user-attachments/assets/e28bb896-51f5-4faa-ab88-5950215429e9" /> --------- Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com> Co-authored-by: Stratou <stratoula1@gmail.com>




Summary
#234488