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

chore(integration): remove deprecated schemas field #522

Conversation

jvallesm
Copy link
Collaborator

Because

  • The schemas field was added under the wrong assumption that the
    integration schemas would differ across connection methods (OAuth vs
    key-value).
  • After deprecating this field and providing an alternative setupSchema
    field, all the clients for the deprecated one have been removed.
    Therefore, removing the deprecated field won't break any client.

This commit

  • Removes the deprecated schemas field.

Copy link

linear bot commented Nov 27, 2024

Copy link

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedNov 27, 2024, 8:29 AM

@jvallesm jvallesm merged commit 03e0861 into main Nov 27, 2024
@jvallesm jvallesm deleted the jvalles/ins-6742-deprecate-the-schemas-usage-of-integration-in-the-console branch November 27, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants