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] Add collapsible sections to 8.0 breaking changes [Part 4] #56356

Merged
merged 8 commits into from
May 11, 2020
Merged

[DOCS] Add collapsible sections to 8.0 breaking changes [Part 4] #56356

merged 8 commits into from
May 11, 2020

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented May 7, 2020

Adds collapsible sections to the 8.0 breaking changes.

To make reviews easier and avoid introducing errors, I've
tried to limit my changes to formatting, with the exception
of rewording heading as sentences.

I'm also splitting this effort over several PRs to make reviews
manageable.

Relates to #55629 and #53229

Preview

http://elasticsearch_56356.docs-preview.app.elstc.co/guide/en/elasticsearch/reference/master/breaking-changes-8.0.html

@jrodewig jrodewig added >docs General docs changes :Docs v8.0.0 labels May 7, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label May 7, 2020
Comment on lines 28 to 31
*Impact* +
Use the available constants (e.g. Fuzziness.ONE, Fuzziness.AUTO) or build your
own instance using the above mentioned factory methods. Use only allowed
`Fuzziness` values.
Copy link
Contributor Author

@jrodewig jrodewig May 7, 2020

Choose a reason for hiding this comment

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

@cbuescher Do you mind confirming that this accurately reflects your original guidance?

This is mostly a formatting change. I just want to ensure I don't unintentionally remove any context here.

Feel free to ignore the other changes in this PR unless you'd like to review them.

Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

Just a couple minor things.

docs/reference/migration/migrate_8_0/java.asciidoc Outdated Show resolved Hide resolved
docs/reference/migration/migrate_8_0/node.asciidoc Outdated Show resolved Hide resolved
@jrodewig jrodewig requested a review from debadair May 7, 2020 16:10
Copy link
Contributor

@debadair debadair left a comment

Choose a reason for hiding this comment

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

LGTM

@jrodewig
Copy link
Contributor Author

To go ahead and get the collapsible sections in, I've reverted most of the changes to the Java API changes section. I'll open a separate PR to get the impact sections of those changes approved.

@jrodewig jrodewig merged commit 303bed8 into elastic:master May 11, 2020
@jrodewig jrodewig deleted the docs__collapse-breaking-changes-pt-4 branch May 11, 2020 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes Team:Docs Meta label for docs team v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants