-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[DOCS] Fixes title out of sequence warnings in breaking changes #63213
Conversation
Pinging @elastic/kibana-docs (Team:Docs) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
||
Move all of the search service code to Kibana new platform. Also to telete `courier` folder | ||
and move `getFlattenedObject` to `src/core/utils`. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change "Also to telete" to "Also to delete"
docs/migration/migrate_7_7.asciidoc
Outdated
[discrete] | ||
==== Add addInfo toast to core notifications service | ||
|
||
Added an Info toast to the core notifications service. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add " This Info toast will be used for the async search notifications"
docs/migration/migrate_7_7.asciidoc
Outdated
Delete any items that are not applicable to this PR. | ||
* Delete `FilterStateManager` | ||
* Delete `QueryFilter` | ||
* Delete `SavedQuery` re-export from legacy |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change to "Delete unused legacy exports FilterStateManager
, QueryFilter
, and SavedQuery
"
LGTM |
…tic#63213) * [DOCS] Add discrete attributes * [DOCS] Addresses title out of sequence warnings * [DOCS] Fixes some formatting in doc * [DOCS] More formatting changes * [DOCS] More clean-up of breaking changes docs * [DOCS] Adds more developer breaking changes * [DOCS] Incorporates review comments * [DOCS] Fixes formatting Co-authored-by: gchaps <chappell_5@yahoo.com>
…) (#63749) * [DOCS] Add discrete attributes * [DOCS] Addresses title out of sequence warnings * [DOCS] Fixes some formatting in doc * [DOCS] More formatting changes * [DOCS] More clean-up of breaking changes docs * [DOCS] Adds more developer breaking changes * [DOCS] Incorporates review comments * [DOCS] Fixes formatting Co-authored-by: gchaps <chappell_5@yahoo.com> Co-authored-by: Lisa Cawley <lcawley@elastic.co>
Summary
Related to #63212 and a6c0f4f
This PR fixes the "title out of sequence" warnings in the 7.7 breaking changes content, and adds "discrete" attributes so that the new content is properly embedded.
It is a work in progress, however, because the content seems to be poorly formatted. I think some of the poorly formatted links, for example, are due to differences between Markdown and Asciidoc conventions.
Checklist
Preview: http://kibana_63213.docs-preview.app.elstc.co/guide/en/kibana/7.x/breaking-changes-7.7.html