-
Notifications
You must be signed in to change notification settings - Fork 781
API Proposal - Client Interceptors #555
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
Comments
@rogchap This is a great start! Thanks for making this big effort to get this feature started. Community, and users of gRPC-Web: please also review this to see if you have any feedback to the API design. Many people have asked about this before. Now is the time to speak up. |
This is amazing! Thanks for working on this! I'll have a closer look at the proposed API next week, but it seems like this is a very good start. |
Yes, agreed. Having a PR with a |
Easily done; I have the .md file of my original write up on disk. Wasn't sure where to submit to get the best visibility for the community; if the gRPC/proposal repo is the right one I'll submit a PR there |
I think we can do it two phases: first we can get one PR here in this repo, then with enough review/revision, we can do one over at grpc/proposal for formality. |
@stanley-cheung @Yannic Moving this to #558 |
gRPC-Web Client InterceptorsProposal move to #558
Previous description removed to avoid confusion
The text was updated successfully, but these errors were encountered: