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

Unify the strings: 'Approve' or 'Proofread'? #719

Closed
hwhsu1231 opened this issue Feb 2, 2024 · 1 comment · Fixed by #791
Closed

Unify the strings: 'Approve' or 'Proofread'? #719

hwhsu1231 opened this issue Feb 2, 2024 · 1 comment · Fixed by #791

Comments

@hwhsu1231
Copy link

Suggestion

I noticed that the subcommands of crowdin status are:

COMMANDS:
  translation   Show only translation progress for a project
  proofreading  Show only proofreading progress for a project

However, the outputs of crowdin status are:

[OK] Fetching project info
Translated:
        - zh-CN: 0%
        - zh-TW: 0%
Approved:
        - zh-CN: 0%
        - zh-TW: 0%

We can see that the Approved is corresponding to proofreading.

I wonder why not unify these strings, thoroughly? (I prefer proofread)

That is, replace the outputs of crowdin status with:

[OK] Fetching project info
Translated:
        - zh-CN: 0%
        - zh-TW: 0%
Proofread:
        - zh-CN: 0%
        - zh-TW: 0%

Versions

crowdin-cli version: 3.16.1

@andrii-bodnar andrii-bodnar linked a pull request May 14, 2024 that will close this issue
@andrii-bodnar
Copy link
Member

Will be available in the v4 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants