Skip to content

[7.9] [Monaco] Refactor the way XJSON grammar checker gets registered (#75160)#75291

Merged
jloleysens merged 2 commits intoelastic:7.9from
jloleysens:backport/7.9/pr-75160
Aug 19, 2020
Merged

[7.9] [Monaco] Refactor the way XJSON grammar checker gets registered (#75160)#75291
jloleysens merged 2 commits intoelastic:7.9from
jloleysens:backport/7.9/pr-75160

Conversation

@jloleysens
Copy link
Copy Markdown
Contributor

Backports the following commits to 7.9:

…tic#75160)

* Refactor the way XJSON grammar checker gets registered

- avoid registering multiple model add listeners
- remove regsiterGrammarChecker from public API!
- fix getWorker handler to register XJSON only for XJSON models

* remove unused import

* updateAnnos -> updateAnnotations
# Conflicts:
#	x-pack/plugins/ingest_pipelines/public/application/components/pipeline_processors_editor/components/manage_processor_form/field_components/xjson_editor.tsx
@jloleysens jloleysens added the backport This PR is a backport of another PR label Aug 18, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

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

@jloleysens jloleysens merged commit 7edb74c into elastic:7.9 Aug 19, 2020
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.

2 participants