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.

Sign up for GitHub

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

Jump to bottom

Document removal of custom REST wrappers in 8.0 #83565

Merged
merged 2 commits into from
Feb 8, 2022
Merged

Document removal of custom REST wrappers in 8.0 #83565

merged 2 commits into from
Feb 8, 2022

Conversation

tvernum
Copy link
Contributor

@tvernum tvernum commented Feb 7, 2022

In #71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section

@tvernum
Document removal of custom REST wrappers in 8.0
a6c9a5e 
In elastic#71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section
@tvernum tvernum added >docs General docs changes :Core/Infra/Plugins Plugin API and infrastructure v8.0.1 v8.2.0 v8.1.1 labels Feb 7, 2022
@elasticmachine elasticmachine added Team:Docs Meta label for docs team Team:Core/Infra Meta label for core/infra team labels Feb 7, 2022
@elasticmachine
Copy link
Collaborator

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

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

bytebilly
bytebilly approved these changes Feb 7, 2022
View reviewed changes
Copy link
Contributor

@bytebilly bytebilly left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@lockewritesdocs
Copy link
Contributor

@elasticmachine test this please

lockewritesdocs
lockewritesdocs reviewed Feb 7, 2022
View reviewed changes
Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

Added some non-blocking comments that you can incorporate if wanted. Otherwise, LGTM!

docs/reference/migration/migrate_8_0/plugin-changes.asciidoc Outdated Show resolved Hide resolved
docs/reference/migration/migrate_8_0/plugin-changes.asciidoc Outdated Show resolved Hide resolved
docs/reference/migration/migrate_8_0/plugin-changes.asciidoc Outdated Show resolved Hide resolved
docs/reference/migration/migrate_8_0/plugin-changes.asciidoc Outdated Show resolved Hide resolved
@tvernum tvernum added auto-backport-and-merge auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Feb 8, 2022
@elasticsearchmachine elasticsearchmachine merged commit ff9c827 into elastic:master Feb 8, 2022
@tvernum tvernum deleted the docs/rest-handler-wrapper branch February 8, 2022 00:47
@tvernum tvernum mentioned this pull request Feb 8, 2022
Merged
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Feb 8, 2022
@tvernum
Document removal of custom REST wrappers in 8.0 (elastic#83565)
f29e721 
In elastic#71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section
@tvernum tvernum mentioned this pull request Feb 8, 2022
Merged
tvernum added a commit to tvernum/elasticsearch that referenced this pull request Feb 8, 2022
@tvernum
Document removal of custom REST wrappers in 8.0 (elastic#83565)
8f73d1f 
In elastic#71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
✅ 8.0
✅ 8.1

elasticsearchmachine pushed a commit that referenced this pull request Feb 8, 2022
@tvernum
Document removal of custom REST wrappers in 8.0 (#83565) (#83623)
3a88013 
In #71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section
elasticsearchmachine pushed a commit that referenced this pull request Feb 8, 2022
@tvernum
Document removal of custom REST wrappers in 8.0 (#83565) (#83622)
27fc17f 
In #71061 we removed support for custom REST Handler Wrappers.

This change adds that information to the migration guide under the
"Plugin changes" section
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lockewritesdocs lockewritesdocs lockewritesdocs left review comments

@bytebilly bytebilly bytebilly approved these changes

Assignees
No one assigned
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Plugins Plugin API and infrastructure >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team v8.0.1 v8.1.1 v8.2.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@tvernum @elasticmachine @lockewritesdocs @elasticsearchmachine @bytebilly