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

Implement custom Marshaler support, add jsonpb implemention. #144

Merged
merged 7 commits into from
May 9, 2016

Conversation

tmc
Copy link
Collaborator

@tmc tmc commented May 7, 2016

This extends upon the work in #130 but keeps net/context conventions in place and updates tests to reflect all the changes.

Addresses #79

@yugui
Copy link
Member

yugui commented May 9, 2016

Thank you, @willtrking, @tmc .
It looks good to me as an initial implementation. I'll add some small fixes later.

@yugui yugui merged commit 3772744 into grpc-ecosystem:master May 9, 2016
@tamird
Copy link
Contributor

tamird commented May 9, 2016

This might also enable fixing #124

@willtrking
Copy link
Contributor

@tamird Yes it does allow that, we did that for a bit in development but never in production.

@tmc tmc deleted the marshaler branch November 20, 2016 21:21
adasari pushed a commit to adasari/grpc-gateway that referenced this pull request Apr 9, 2020
Implement custom Marshaler support, add jsonpb implemention.
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.

4 participants