Skip to content
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

Improve zh-tw (Traditional Chinese) locale #18608

Merged
merged 1 commit into from
May 8, 2023

Conversation

PeterDaveHello
Copy link
Contributor

@PeterDaveHello PeterDaveHello commented Apr 26, 2023

Improve zh-tw (Traditional Chinese) locale

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

@pichuang
Copy link

As a long-time user of the zh-tw language, I personally recommend this PR.

@codecov
Copy link

codecov bot commented Apr 27, 2023

Codecov Report

Merging #18608 (0f541a2) into main (aaea068) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head 0f541a2 differs from pull request most recent head 8e622f0. Consider uploading reports for the commit 8e622f0 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #18608      +/-   ##
==========================================
- Coverage   67.39%   67.37%   -0.02%     
==========================================
  Files         984      984              
  Lines      107029   107029              
  Branches     2670     2670              
==========================================
- Hits        72130    72110      -20     
- Misses      31014    31042      +28     
+ Partials     3885     3877       -8     
Flag Coverage Δ
unittests 67.37% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 12 files with indirect coverage changes

@AllForNothing
Copy link
Contributor

@PeterDaveHello Thanks for your contribution.
Handling of keys and parameters looks ok. Can you invite someone to help to review your translation?

@wy65701436
Copy link
Contributor

Thank you, @pichuang, for your comments. Please go ahead and add an LGTM. Additionally, we would appreciate another community member who uses the zh-tw language to review it. By the way, I am generally satisfied with these changes.

Copy link
Contributor

@wy65701436 wy65701436 left a comment

Choose a reason for hiding this comment

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

thanks, LGTM

Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

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

lgtm

"WAITING_TOO_LONG_3": "For more details, please refer to the ",
"WAITING_TOO_LONG_4": "Wiki."
"WAITING_TOO_LONG_1": "某些工作等待執行超過 24 小時。請檢查工作服務儀表板。",
"WAITING_TOO_LONG_2": "更多詳細資訊,請參考 Wiki。",
Copy link
Contributor

Choose a reason for hiding this comment

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

"dashboard" and "Wiki" are hyperlink text, so they should be stand-alone items

Copy link
Contributor

@AllForNothing AllForNothing May 5, 2023

Choose a reason for hiding this comment

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

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, thanks.

"SCHEDULE": "排程",
"TEST_FAILED": "測試失敗",
"MANUAL": "手動",
"SCHEDULED": "已排程",
Copy link
Contributor

Choose a reason for hiding this comment

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

This item is used to describe the task type.
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated, thanks.

Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
@AllForNothing AllForNothing merged commit 0e3e70b into goharbor:main May 8, 2023
@PeterDaveHello PeterDaveHello deleted the zhtw branch May 9, 2023 05:52
WilfredAlmeida pushed a commit to WilfredAlmeida/harbor that referenced this pull request Jul 8, 2023
Improve zh-tw locale

Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
Signed-off-by: Wilfred Almeida <60785452+WilfredAlmeida@users.noreply.github.com>
PeterDaveHello added a commit to PeterDaveHello/harbor that referenced this pull request Aug 13, 2023
Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
@PeterDaveHello
Copy link
Contributor Author

Hey guys, I sent a follow up of this PR - #19161 to continue the improvement works of zh-tw locale here, would be great to have your reviews, thanks.

PeterDaveHello added a commit to PeterDaveHello/harbor that referenced this pull request Aug 13, 2023
Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
PeterDaveHello added a commit to PeterDaveHello/harbor that referenced this pull request Aug 13, 2023
Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
PeterDaveHello added a commit to PeterDaveHello/harbor that referenced this pull request Aug 14, 2023
Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
PeterDaveHello added a commit to PeterDaveHello/harbor that referenced this pull request Aug 15, 2023
Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
AllForNothing pushed a commit that referenced this pull request Aug 16, 2023
Update zh-tw (Traditional Chinese) locale, follow up of #18608

Signed-off-by: Peter Dave Hello <hsu@peterdavehello.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/i18n Related to Translation changes. release-note/update Update or Fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants