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

Remove Zenpy usage for Tickets stream #75

Merged
merged 22 commits into from
Oct 5, 2021

Conversation

jacobrobertbaca
Copy link
Contributor

@jacobrobertbaca jacobrobertbaca commented Sep 30, 2021

Description of change

https://jira.talendforge.org/browse/TDL-15551

Fixing the via.from field on the tickets, and ticket_comments stream.

The previous version bump on the zenpy library changed how the zenpy was returning the via.from field to via._from. To work around this we are dropping the usage of the zenpy library for the tickets stream and all children streams.

Manual QA steps

  • tests will run in circle
  • that's it for now

Risks

  • if we implement the sync and pagination correctly, there should be no risks

Rollback steps

  • revert this branch

@cosimon cosimon changed the title Fix/query for tickets not using zenpy Remove Zenpy usage for Tickets stream Oct 1, 2021
@cosimon cosimon merged commit 88fdfec into master Oct 5, 2021
@cosimon cosimon deleted the fix/query-for-tickets-not-using-zenpy branch October 5, 2021 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants