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

revert(api): remove /plugins/schema/:name #11313

Merged
merged 1 commit into from
Jul 28, 2023
Merged

Conversation

kikito
Copy link
Member

@kikito kikito commented Jul 27, 2023

Partially reverts #10846

Even though we have been emitting deprecation warnings when the endpoint
was used, we didn't remove it in 3.0, and doing it now can be a surprise
breaking change for someone. In fact, someone had to update a project
which was relying on that path, that is linked in #10846

This reverts the removal and replaces the deprecation version with 4.0
instead of 3.0. This PR does not revert the changes in our tests to use
the new endpoint instead of the deprecated one.

Summary

Checklist

Full changelog

  • [Implement ...]

Issue reference

Fix #[issue number]

@kikito kikito added this to the 3.4.0 milestone Jul 27, 2023
@kikito kikito force-pushed the revert-plugins-schema-removal branch from 08f7ac6 to c790002 Compare July 27, 2023 18:46
Partially reverts #10846

Even though we have been emitting deprecation warnings when the endpoint
was used, we didn't remove it in 3.0, and doing it now can be a surprise
breaking change for someone. In fact, someone had to update a project
which was relying on that path, that is linked in #10846

This reverts the removal and replaces the deprecation version with 4.0
instead of 3.0. This PR does not revert the changes in our tests to use
the new endpoint instead of the deprecated one.
@kikito kikito force-pushed the revert-plugins-schema-removal branch from c790002 to fcab335 Compare July 27, 2023 18:51
@flrgh flrgh merged commit 1ec4d73 into master Jul 28, 2023
@flrgh flrgh deleted the revert-plugins-schema-removal branch July 28, 2023 17:34
team-gateway-bot pushed a commit that referenced this pull request Jul 28, 2023
Partially reverts #10846

Even though we have been emitting deprecation warnings when the endpoint
was used, we didn't remove it in 3.0, and doing it now can be a surprise
breaking change for someone. In fact, someone had to update a project
which was relying on that path, that is linked in #10846

This reverts the removal and replaces the deprecation version with 4.0
instead of 3.0. This PR does not revert the changes in our tests to use
the new endpoint instead of the deprecated one.

(cherry picked from commit 1ec4d73)
flrgh pushed a commit that referenced this pull request Jul 28, 2023
Partially reverts #10846

Even though we have been emitting deprecation warnings when the endpoint
was used, we didn't remove it in 3.0, and doing it now can be a surprise
breaking change for someone. In fact, someone had to update a project
which was relying on that path, that is linked in #10846

This reverts the removal and replaces the deprecation version with 4.0
instead of 3.0. This PR does not revert the changes in our tests to use
the new endpoint instead of the deprecated one.

(cherry picked from commit 1ec4d73)

Co-authored-by: Enrique García Cota <kikito@gmail.com>
@kikito kikito mentioned this pull request Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants