Skip to content

[8.6] [PresentationUtil] Fix Canvas expression autocomplete (#146425)#146465

Merged
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-146425
Nov 28, 2022
Merged

[8.6] [PresentationUtil] Fix Canvas expression autocomplete (#146425)#146465
kibanamachine merged 1 commit intoelastic:8.6from
kibanamachine:backport/8.6/pr-146425

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.6:

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Nov 28, 2022
@kibanamachine kibanamachine enabled auto-merge (squash) November 28, 2022 19:36
Fixes elastic#146243

## Summary

Fixes Canvas expression autocomplete

elastic#143739 upgraded the monaco-editor
dependency which uses a callback to the `onLanguage` method to
initialize the expressions. The PR moved the `monaco.languages.register`
command inside this callback and which was never triggered.

Moving the `monaco.languages.register` command outside the callback
appears to fix the issue.

(cherry picked from commit 19413b7)
@kibana-ci
Copy link

💚 Build Succeeded

Metrics [docs]

Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 59 65 +6
osquery 108 113 +5
securitySolution 441 447 +6
total +19

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 68 74 +6
osquery 109 115 +6
securitySolution 518 524 +6
total +20

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

cc @nickpeihl

@kibanamachine kibanamachine merged commit 301bd83 into elastic:8.6 Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants