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

[DOCS] Fixes title out of sequence warnings in breaking changes #63213

Merged
merged 8 commits into from
Apr 16, 2020

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Apr 10, 2020

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

  • Documentation was added for features that require explanation or tutorials

Preview: http://kibana_63213.docs-preview.app.elstc.co/guide/en/kibana/7.x/breaking-changes-7.7.html

@lcawl lcawl added Team:Docs WIP Work in progress release_note:skip Skip the PR/issue when compiling release notes v7.8.0 labels Apr 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

Copy link
Contributor

@gchaps gchaps left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gchaps gchaps added v7.7.0 and removed WIP Work in progress labels Apr 13, 2020
@gchaps gchaps requested a review from jinmu03 April 13, 2020 20:54
@gchaps gchaps marked this pull request as ready for review April 13, 2020 21:00

Move all of the search service code to Kibana new platform. Also to telete `courier` folder
and move `getFlattenedObject` to `src/core/utils`.

Copy link
Contributor

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"

[discrete]
==== Add addInfo toast to core notifications service

Added an Info toast to the core notifications service.
Copy link
Contributor

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"

Delete any items that are not applicable to this PR.
* Delete `FilterStateManager`
* Delete `QueryFilter`
* Delete `SavedQuery` re-export from legacy
Copy link
Contributor

@jinmu03 jinmu03 Apr 13, 2020

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"

@jinmu03
Copy link
Contributor

jinmu03 commented Apr 16, 2020

LGTM

@gchaps gchaps merged commit 0f39ea6 into elastic:7.x Apr 16, 2020
@gchaps gchaps deleted the 7x-plugin-changes branch April 16, 2020 18:45
gchaps added a commit to gchaps/kibana that referenced this pull request Apr 16, 2020
…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>
gchaps added a commit that referenced this pull request Apr 16, 2020
…) (#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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes Team:Docs v7.7.0 v7.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants