Skip to content

[WIP] Feature/protocol json #8

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

Closed
wants to merge 11 commits into from
Closed

[WIP] Feature/protocol json #8

wants to merge 11 commits into from

Conversation

jeskew
Copy link
Contributor

@jeskew jeskew commented May 23, 2017

For now, this includes Base64 (browser/node/universal), UTF-8 (browser/node/universal), timestamp formatting, and a JSON body marshaller. Still working on the wrapper AWS JSON marshaller/unmarshaller.

@jeskew
Copy link
Contributor Author

jeskew commented May 27, 2017

This got a little too big for a single PR due to the number of dependent tasks that needed to be completed for a marshaller to function properly, so closing in favor of #14, #13, #12, #11, and #10, each of which can be reviewed and incorporated independently.

@jeskew jeskew closed this May 27, 2017
@jeskew jeskew deleted the feature/protocol-json branch November 14, 2017 21:26
@lock
Copy link

lock bot commented Sep 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.

@lock lock bot locked as resolved and limited conversation to collaborators Sep 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant