-
Notifications
You must be signed in to change notification settings - Fork 41
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
Refactored the 2.0.0 changelog #192
Conversation
@@ -10,29 +10,24 @@ This project adheres to [Semantic Versioning](http://semver.org/). | |||
- An optional "stack identifier" that can be set during [initialization](README.md#initialization) so we can track libraries that use node-sparkpost via the User-Agent header by @ewandennis. | |||
|
|||
## [2.0.0] - 2016-11-04 - *breaking* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it might be worth adding a human-readable paragraph here summarizing the changes and that it's a major version release, etc.
- [#112](https://github.com/SparkPost/node-sparkpost/pull/112) Returns body.results or body as a response. Added debug support. (@aydrian) | ||
### Added | ||
- Support for Promises and Callbacks | ||
- Debug option on initialization attached debug information on response |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/attached/attaches
- Debug option on initialization attached debug information on response | ||
|
||
### Changed | ||
- Methods return `data` comprised of the response body. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It isn't clear to me what this means. Maybe an example of what this looks like or a link to an example?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wasn't sure how to say response.body
is returned instead of response
|
||
### Changed | ||
- Methods return `data` comprised of the response body. | ||
- Standardized methods on all API wrappers. See Issue #175. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add link to issue 175
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's supposed to do that automatically according to the Github markdown guide.
- Now using NPM scripts instead of grunt | ||
|
||
### Fixed | ||
- Responses for `GET` requests are now properly parsed as JSON by @aydrian. Closes #111 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Link to issue 111
Going forward, we will use Keep a Changelog as a guide.
Addresses #191