Skip to content

find admin by id fixes #159#165

Merged
jkeyes merged 1 commit intointercom:masterfrom
jD0reenWerner:find-admin-by-id
Oct 24, 2017
Merged

find admin by id fixes #159#165
jkeyes merged 1 commit intointercom:masterfrom
jD0reenWerner:find-admin-by-id

Conversation

@jD0reenWerner
Copy link
Copy Markdown
Contributor

So far, there is only the method intercom.admins.all() to list all admins.
But Intercom also offers to find an admin by ID.

Use it like this:
admin = intercom.admins.find(id='12345')

@coveralls
Copy link
Copy Markdown

coveralls commented Jun 12, 2017

Coverage Status

Coverage increased (+0.002%) to 95.833% when pulling 46c9107 on jgiebelhausen:find-admin-by-id into abcc272 on jkeyes:master.

@raphael-riel
Copy link
Copy Markdown

I confirm I needed this to access an admin's Avatar picture.
Adding the Find trait to Admin was my solution too. So this PR gets my seal of approval 😄

@jkeyes jkeyes merged commit 1bbe2f7 into intercom:master Oct 24, 2017
@jkeyes
Copy link
Copy Markdown
Contributor

jkeyes commented Oct 24, 2017

Thanks @jgiebelhausen 👍

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.

4 participants