-
Notifications
You must be signed in to change notification settings - Fork 157
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
add link at Supporters redirect to the instructions doc. #482
Conversation
94ae28f
to
c0e4392
Compare
}, | ||
"item.label.Next": { | ||
"message": "Next", | ||
"description": "Navbar item with label Next" | ||
}, | ||
"item.label.v1.15": { | ||
"message": "v1.15", | ||
"description": "Navbar item with label v1.15" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another question, what's meaning of labels above? And what's the effect without them?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another question, what's meaning of labels above? And what's the effect without them?
Auto generated Chinese version of the label name using yarn write-translations
. Actually, when we do any changes on the pages, we need to using yarn write-translations --locale zh
to generate the corresponding Chinese version and translate it.
For example, if I change the "message" content to "1.15 版本" now, it would be displayed in Chinese as follows.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, sounds good, if it's auto generated, how about adding a check in CI?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, sounds good, if it's auto generated, how about adding a check in CI?
Maybe it's worth to add this CI to check the Chinese version. I will create a issue to track.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okey. Let's open an issue to track it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okey. Let's open an issue to track it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
/assign @kevin-wangzefeng
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fisherxu The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: Shelley-BaoYue <[email protected]>
Signed-off-by: Shelley-BaoYue <[email protected]>
9b1c18b
to
985a535
Compare
Made a small optimization, the whole title of "Join the growing community" can support link redirection. PTAL @fisherxu @kevin-wangzefeng |
985a535
to
0f37656
Compare
Signed-off-by: Shelley-BaoYue <[email protected]>
0f37656
to
fd444fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
add link at Supporters redirect to the instructions doc. Signed-off-by: Shelley-BaoYue <[email protected]>
feature
Add a link at Supporters to help users to get "how to add logo at supporters"
#470