Skip to content
This repository has been archived by the owner on Jan 2, 2022. It is now read-only.

Added basic support for gwtjsonrpc #8

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Added basic support for gwtjsonrpc #8

wants to merge 5 commits into from

Conversation

ericbets
Copy link

Added basic support for gwtjsonrpc.

@RitwikSaikia
Copy link
Owner

I had a query, regarding this patch.

gwtjsonrpc is a complete rpc server/client implementation. Why do we want to mix it with this project ?

If providing a GWT client is the only reason, that is in the road map, and will be implemented.

@ericbets
Copy link
Author

The wire protocol for a GWT client is purposefully vague so that it can be modified later. Using gerrit's gwtjsonrpc provides an alternative to Google's App engine tooling, with a wire protocol that is explicit.

@ericbets
Copy link
Author

By the way, the gwtjsonrpc client only works from the browser.

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.

3 participants