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

Use correct PHPdoc types for user IDs #201

Merged
merged 2 commits into from
Apr 4, 2017
Merged

Use correct PHPdoc types for user IDs #201

merged 2 commits into from
Apr 4, 2017

Conversation

fnwbr
Copy link
Contributor

@fnwbr fnwbr commented Feb 22, 2017

IDs are strings, not integers: https://developers.intercom.com/reference#user-model
Also, get() receives the $query parameters as an array, not as a string.

@brianw
Copy link
Contributor

brianw commented Apr 4, 2017

Sorry for the slow response here 😞 Thank you so much for correcting our PHPdoc.

I'll merge this change now, it'll be included in the next patch release of the client which should be early next week.

@brianw brianw merged commit 9ebffdf into intercom:master Apr 4, 2017
@fnwbr fnwbr deleted the patch-1 branch April 4, 2017 15:38
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