Skip to content

zh-translation: docs/root/configuration/http/http_filters/rate_limit_… - #636

Merged
helight merged 1 commit into
cloudnativeto:zhfrom
wujie1993:configuration_rate_limit_filter
Mar 24, 2021
Merged

zh-translation: docs/root/configuration/http/http_filters/rate_limit_…#636
helight merged 1 commit into
cloudnativeto:zhfrom
wujie1993:configuration_rate_limit_filter

Conversation

@wujie1993

Copy link
Copy Markdown
Member

issue #296

@wujie1993
wujie1993 force-pushed the configuration_rate_limit_filter branch 5 times, most recently from 558b9aa to d8ce6c2 Compare January 27, 2021 01:44

@addozhang addozhang left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

翻译的很好,几个小建议。

Comment thread docs/root/configuration/http/http_filters/rate_limit_filter.rst Outdated
Comment thread docs/root/configuration/http/http_filters/rate_limit_filter.rst Outdated
@majinghe

majinghe commented Feb 2, 2021

Copy link
Copy Markdown

@wujie1993 rate limit 统一翻译成限流。

@wujie1993
wujie1993 force-pushed the configuration_rate_limit_filter branch from ac6d666 to a9541f8 Compare February 4, 2021 07:55
@wujie1993

Copy link
Copy Markdown
Member Author

好的,已按照建议内容更新

@majinghe

majinghe commented Feb 5, 2021

Copy link
Copy Markdown

/LGTM

@majinghe

majinghe commented Feb 5, 2021

Copy link
Copy Markdown

@addozhang 辛苦晓辉,看一下这个 PR。

…filter.rst

Signed-off-by: wujie1993 <qq594jj@gmail.com>
Co-authored-by: Addo.Zhang <duwasai@gmail.com>
@wujie1993
wujie1993 force-pushed the configuration_rate_limit_filter branch from a9541f8 to f41acf6 Compare February 5, 2021 01:54
@helight
helight self-requested a review March 23, 2021 11:57

@helight helight left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@helight
helight merged commit a457080 into cloudnativeto:zh Mar 24, 2021
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