Skip to content

[api] Add user CRUD#89

Merged
yannmh merged 1 commit intomasterfrom
iddl/user-crud
Oct 19, 2015
Merged

[api] Add user CRUD#89
yannmh merged 1 commit intomasterfrom
iddl/user-crud

Conversation

@iddl
Copy link
Copy Markdown

@iddl iddl commented Oct 14, 2015

Additional changes:

  • Add a deprecation message to invite method of api/users
  • 409 HTTP error codes are now recognized as ApiError exceptions

Additional changes:
- Add a deprecation message to `invite` method of `api/users`
- 409 HTTP error codes are now recognized as ApiError exceptions
@remh
Copy link
Copy Markdown

remh commented Oct 16, 2015

@yannmh for you

@yannmh
Copy link
Copy Markdown

yannmh commented Oct 19, 2015

Changes look great ! Thanks @iddl.

Let's merge and release a new version today 🚢 !

yannmh pushed a commit that referenced this pull request Oct 19, 2015
@yannmh yannmh merged commit 5960876 into master Oct 19, 2015
@yannmh
Copy link
Copy Markdown

yannmh commented Oct 19, 2015

On a side note, the API documentation needs an update. @iddl do you know if anyone is on it ?

@yannmh yannmh deleted the iddl/user-crud branch October 19, 2015 15:43
@iddl
Copy link
Copy Markdown
Author

iddl commented Oct 19, 2015

@technovangelist will proceed with the PR review as soon as the library changes are upstream DataDog/documentation#521

@yannmh
Copy link
Copy Markdown

yannmh commented Oct 19, 2015

datadog==0.10.0 is out. Thanks again.

@remh
Copy link
Copy Markdown

remh commented Oct 19, 2015

🎉

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.

3 participants