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

Document parameter alert_type in Event.create #110

Merged
merged 1 commit into from
Jan 5, 2016

Conversation

aknuds1
Copy link
Contributor

@aknuds1 aknuds1 commented Jan 5, 2016

As far as I can tell, you've forgot to document the parameter alert_type in Event.create. I guess there should also be a test for this, but I wasn't sure how to go about this.

@yannmh yannmh self-assigned this Jan 5, 2016
@yannmh
Copy link
Member

yannmh commented Jan 5, 2016

You're right alert_type is a valid input as listed in the API documentation.
Thanks for adding it @aknuds1 📝 !

About testing, this could be the subject of a small addition in our integration test suite ✅. I'll make the according changes in a separate pull request.

yannmh added a commit that referenced this pull request Jan 5, 2016
Document parameter alert_type in Event.create
@yannmh yannmh merged commit 8928bcb into DataDog:master Jan 5, 2016
@aknuds1
Copy link
Contributor Author

aknuds1 commented Jan 5, 2016

@yannmh Thanks. Yeah, I was going to add it to the integration test suite but I could not see how to run it :(

@yannmh yannmh added this to the Next milestone Jan 8, 2016
@yannmh yannmh modified the milestones: 0.11.0, Next Mar 10, 2016
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.

2 participants