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

Removing empy zh_Han* files that are raising an alert in Weblate #2890

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

ctlaltdieliet
Copy link
Collaborator

Summary

With moving the WIP-languages to the shipped project, we got duplicate files for Traditional and Simplified Chinese.
Although zh_Hans and zh_Hant are more correct, it would require code-changes and existing Chinese users to chose their language again.
As these filenames are not visible, we can keep using these

Screenshots

mattermost-desktop

Release Note

NONE

@mattermost-build
Copy link
Contributor

Hello @ctlaltdieliet,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thank you, @ctlaltdieliet!

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@devinbinnie
Copy link
Member

@ctlaltdieliet Am I good to merge this?

@ctlaltdieliet
Copy link
Collaborator Author

@devinbinnie yes this is safe to merge!

@devinbinnie devinbinnie merged commit 7aed1eb into master Nov 9, 2023
@devinbinnie devinbinnie deleted the i18n-removing-duplicate-cn-translation-files branch November 9, 2023 16:24
@amyblais amyblais added this to the v5.6.0 milestone Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants