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

✨ [source-zendesk-support] Fix the fields data type to include the id and value #36388

Merged

Conversation

mohamagdy
Copy link
Contributor

@mohamagdy mohamagdy commented Mar 22, 2024

Set the JSON data type type of fields field items in Zendesk Support Tickets API

Copy link

vercel bot commented Mar 22, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 25, 2024 0:51am

@marcosmarxm marcosmarxm changed the title Fix the fields data type to include the id and value ✨ [source-zendesk-support] Fix the fields data type to include the id and value Mar 22, 2024
@marcosmarxm
Copy link
Member

@mohamagdy please update the pull request description. Can you link the Zendesk documentation showing the field is valid? Did you run tests confirming the API actually return these values?

@mohamagdy
Copy link
Contributor Author

@marcosmarxm thanks for taking a look into this PR. Unfortunately, I can't find the fields field in Zendesk documentation here https://developer.zendesk.com/api-reference/ticketing/tickets/tickets/#json-format. And yes, I can see the fields field returned from the API. What I noticed is that the values of the custom_fields and fields are the same (maybe this was for my case only). Maybe the fields and custom_fields are aliases, not sure 😕

@marcosmarxm
Copy link
Member

@mohamagdy any chance to contact Zendesk to make clear this distinction? Would be quite helpful if you could this :)

@mohamagdy
Copy link
Contributor Author

@marcosmarxm I asked Zendesk Customer Service and they pointed me to this page https://support.zendesk.com/hc/en-us/articles/4408886739098-About-ticket-fields saying that the fields field should contain the Standard Fields but the custom_fields should contain the Custom Fields.

Copy link
Member

@marcosmarxm marcosmarxm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small change than ready to go.

@octavia-squidington-iii octavia-squidington-iii added the area/documentation Improvements or additions to documentation label Apr 25, 2024
@lazebnyi lazebnyi self-requested a review April 25, 2024 10:09
Copy link
Collaborator

@lazebnyi lazebnyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

After @marcosmarxm approve can be merged

@lazebnyi lazebnyi enabled auto-merge (squash) April 25, 2024 10:10
@lazebnyi lazebnyi merged commit 135e623 into airbytehq:master Apr 25, 2024
56 of 57 checks passed
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.

4 participants