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

The (new in 0.22.0) articles.contentTagIds field has the wrong Type (is List<Long> - should be List<String> #581

Closed
andy-may-at opened this issue Apr 19, 2023 · 1 comment

Comments

@andy-may-at
Copy link
Contributor

Describe the bug
When I added the field Articles.contentTagIds in this commit, I trusted the example responses in the Zendesk documentation & so added the field as a List<Long>

But it turns out that the articles API docs were wrong & the IDs are really String values (not Long)

Evidence:

The docs for the content_tags resource show it as a String.

And an example response from my Zendesk instance's Articles API has String values:

     "content_tag_ids": [
         "01GMAGH6KPDHEY79MN3S4H8V7D"
     ],

I would expect this bug to cause deserialisation errors when parsing article responses that contain content_tag_ids value

I'll get a PR to fix in ASAP.

@andy-may-at
Copy link
Contributor Author

PR is here: #582

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

No branches or pull requests

1 participant