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

chore: clarify IDs and explain how to create a company #611

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Startouf
Copy link

it can be confusing which "user ID" we are talking about

  • Many users of this intercom library have their own users in their own database, and their "customer ID"
  • Intercom has its own unique user ID

Also, there was no example to create a company.

Add create example
Add information regarding intercom IDs
@Startouf
Copy link
Author

I'm thinking of updating my PR to use "external id" to better match intercom documentation. I also found out one had to use intercom.find(external_id: user_in) and not (user_id: user_id)

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.

1 participant