Hey @jkeyes, I'm working on adding support to set user companies (even if the API doesn't return the information to read yet).

Just wanted to get your thoughts if on what the interaction should be for setting companies (or a company). Since the Intercom API currently allows you to set an array/list of companies, I was thinking something like this?

user = User(email="foo@bar.com")
user.companies = [
    {"id":6, "name": "Intercom", "created_at": datetime.now()}, 
    {"id":7, "name": "GitHub", "created_at": datetime.now()}]

But also, inspired by the ruby client, you could also allow a single company to be set with

user.company = {"id":6, "name": "Intercom", "created_at": datetime.now()}

On that note, since the API doesn't return companies information, do you have any suggestions on what should happen when you try to access the company/companies attribute? I see the convention you've used before is to raise an AttributeError? Would something like this make sense?

user.companies
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError("companies is a write-only property")

user.company
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError("company is a write-only property")

Let me know if that sounds good and I'll go about adding it on my fork! :)

Support for companies. #10

Description

Activity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions