report method has been modified to accept ajaxOption object containin… #158
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Includes JSON3 in the polyfill version Fixes stacktracejs#133
Convert reports to JSON
Add bower dependencies
Member
|
Looks nice 👍 |
Author
|
Thank you, it helps a lot on Restful APIs. |
Member
|
This implementation looks good. I've love to merge this if you wouldn't mind adding a unit test and updating the README to add the optional argument to |
Author
|
yes, it would be great. |
Member
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
a new parameter has been added to report method called ajaxOptions, if the object contains a headers property the headers will be added to the request headers.
eg:
The report method adds the "Authorization" header to the request.