Skip to content

Conversation

@tashiya-menuri
Copy link
Contributor

What this PR does / why we need it:

Fix for #2327

@tashiya-menuri tashiya-menuri changed the title bugfux : fixes issue #2327 doc : fixes issue #2327 Oct 27, 2020
* [Router radixtree](router-radixtree.md)
* [Stand Alone Model](stand-alone.md): Supports to load route rules from local yaml file, it is more friendly such as under the kubernetes(k8s).
* [Stream Proxy](stream-proxy.md)
* [gRPC Proxy](grpc-proxy.md)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to update the Chinese version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed this in latest commit.

@spacewander spacewander added the wait for update wait for the author's response in this issue/PR label Oct 29, 2020
@tashiya-menuri
Copy link
Contributor Author

@moonming @spacewander @membphis Can you please check now?

-->

[中文](zh-cn/grpc-proxy.md)
[Chinese](zh-cn/grpc-proxy.md)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be better if you leave this unchanged.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@membphis
Copy link
Member

@tashiya-menuri one for one thing.

fix doc: add grpc-proxy.md link in doc/README.md #2327 is enough.

@tashiya-menuri
Copy link
Contributor Author

@membphis @spacewander Addressed the requested changes in latest commit. Can you please have a look now?

@membphis membphis merged commit acaae91 into apache:master Oct 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wait for update wait for the author's response in this issue/PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants