search

intercom / intercom-dotnet

Intercom API client library for .NET
https://developers.intercom.io/reference
Apache License 2.0
63 stars 53 forks source link

Enable removal of company from user on update #93

Closed juliancarrivick closed 6 years ago

juliancarrivick commented 6 years ago

Fixes #92. Previously the only way you could remove a user from a company was to use the provided methods. Adding the remove property to the company when updating a user allows this to be done when updating the user, preventing the need for two separate calls. Have not created or updated any tests since there aren't any currently. As such I have reduced my scope of impact by only changing the code which is strictly necessary.

kmossco commented 6 years ago

Perfect, I'll have a look at it later today. Thank you for taking the time to send this over! And we currently don't have any tests on this, so no worries, I need to get to it at some point.