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

Add "update many" to users #415

Merged
merged 1 commit into from
Dec 10, 2019
Merged

Conversation

toru-takahashi
Copy link
Contributor

@toru-takahashi toru-takahashi commented Dec 9, 2019

Hello Team,

First, Thank you for maintaining Zendesk API Ruby client!
Please help me to introduce this fix!

Users have "update many" endpoint.
Ref. https://developer.zendesk.com/rest_api/docs/support/users#update-many-users

But it was removed by this commit; f34990d

So this is a fix to add the endpoint.

As a background, I maintain the following tool to add some user attribution data to our Zendesk User Fields. Then, the latest version doesn't work because the tool uses update_many!
https://github.com/toru-takahashi/embulk-output-zendesk_users

Users have "update many" endpoint.
Ref. https://developer.zendesk.com/rest_api/docs/support/users#update-many-users

But it was removed by this commit; zendesk@f34990d

So this is a fix to add the endpoint.
@nogates
Copy link
Contributor

nogates commented Dec 10, 2019

Thanks for the contribution @toru-takahashi ! I will be releasing a new version soon!

@nogates nogates merged commit c2b85bb into zendesk:master Dec 10, 2019
@toru-takahashi
Copy link
Contributor Author

Thank you for your prompt response and fix!

@toru-takahashi toru-takahashi deleted the patch-1 branch December 15, 2019 03:01
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.

2 participants