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

[api] Add user CRUD #89

Merged
merged 1 commit into from
Oct 19, 2015
Merged

[api] Add user CRUD #89

merged 1 commit into from
Oct 19, 2015

Conversation

iddl
Copy link
Contributor

@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
Contributor

remh commented Oct 16, 2015

@yannmh for you

@yannmh
Copy link
Member

yannmh commented Oct 19, 2015

Changes look great ! Thanks @iddl.

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

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

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
Contributor 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
Member

yannmh commented Oct 19, 2015

datadog==0.10.0 is out. Thanks again.

@remh
Copy link
Contributor

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