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

Update CUSTOM_TYPES to include lookup field #124

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lukas-gust
Copy link
Contributor

@lukas-gust lukas-gust commented Apr 19, 2023

Description of change

This PR adds the lookup custom type to the CUSTOM_TYPES lookup. Lookups are references to other zendesk resources so their values are integers.

https://support.zendesk.com/hc/en-us/articles/4552827736218-Announcing-lookup-relationship-fields-for-Support
https://support.zendesk.com/hc/en-us/articles/4591924111770#topic_wny_w3k_5tb

Relates to #117

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

Manual QA steps

  • Ran sync with zendesk instance that uses lookups on organizations

Risks

Rollback steps

  • revert this branch

+ Added lookup custom field type with type integer.
@lukas-gust
Copy link
Contributor Author

@dan-ladd
Copy link

dan-ladd commented Jun 20, 2023

Thanks @lukas-gust, we started recently running into this issue too and your PR looks good to me. Hoping someone can review and merge?

@JYOTHINARAYANSETTY , you've been on a tear recently with tap-zendesk. Thank you! Would you be able to look at this small PR? Thanks!

@cngpowered cngpowered self-requested a review July 19, 2023 09:14
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