Skip to content

Conversation

crazyzhou
Copy link
Contributor

[FLINK-11560] Translate "Flink Applications" page into Chinese

Copy link

@Tobehalo Tobehalo left a comment

Choose a reason for hiding this comment

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

test

Copy link
Member

@klion26 klion26 left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @crazyzhou I've reviewed to section Time and left some comments.
Please also consider to read the doc for better translate.

Copy link
Member

@klion26 klion26 left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the comments, @crazyzhou I've reviewed the last part of the translation and left some comments.

@wuchong
Copy link
Member

wuchong commented Jun 3, 2019

Hi @crazyzhou @klion26 , what's the status of the pull request now ?

@crazyzhou
Copy link
Contributor Author

crazyzhou commented Jun 6, 2019

@klion26 has reviewed all the text till the end, and I think I've finished all the fixes, just need a final check from @wuchong or @klion26 .

@klion26
Copy link
Member

klion26 commented Jun 24, 2019

LGTM +1

@wuchong
Copy link
Member

wuchong commented Jun 24, 2019

Thanks @klion26 , I will merge it.

@asfgit asfgit closed this in 6a5c25f Jun 24, 2019
XComp pushed a commit to XComp/flink-web that referenced this pull request Sep 7, 2020
curcur pushed a commit to curcur/flink-web that referenced this pull request Apr 5, 2021
snuyanzin pushed a commit to snuyanzin/flink-web that referenced this pull request Feb 3, 2025
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