-
Notifications
You must be signed in to change notification settings - Fork 4.6k
✨ Source Zendesk Support: migration to low-code #56354
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
Conversation
…94/tickets-to-low-code
…94/tickets-to-low-code
…le_streams-to-low-code
…le_streams-to-low-code
…4/prepare-low-code-release
…4/prepare-low-code-release
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Last round of regression tests (all considering state):
EDIT: analysis
user_identities:
tickets:
articles:
article_votes:
ticket_metrics:
Based on this, it seems like we are ready for a release tomorrow where most streams will be considered as a breaking change given the state changes. |
/approve-regression-tests
|
airbyte-integrations/connectors/source-zendesk-support/source_zendesk_support/spec.json
Show resolved
Hide resolved
airbyte-integrations/connectors/source-zendesk-support/unit_tests/integrations/test_articles.py
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
APPROVED
What
This is a stacked PR including all the part for https://github.com/airbytehq/airbyte-internal-issues/issues/11894:
How
See the stacked PRs for more information
Review guide
The changes have been reviewed in the stacked PRs
User Impact
This should speed up some of the streams as they would now run in the concurrent engine.
It also removes
ignore_pagination
.Can this PR be safely reverted and rolled back?
There are some streams where the state changes which means we can't revert.