Skip to content

[l10n] Complete translations #3794

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

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

KristjanESPERANTO
Copy link
Collaborator

Short description: I completed the translations with the help of translation tools.

Long description: I'm currently looking at the translation-tests and I noticed the e2e-test ${language} should contain all base keys, (). It is supposed to check whether all keys are present in each translation file, but it doesn't actually work that way. When I fix it, a lot of missing translations are reported. I have completed these translations with the help of translation tools and packed them into this PR. I have left out the modified test so that we can focus on the question if we accept such amount of automatic translations or not.

rejas has already expressed in another PR (#3775 (comment)) that he prefers human translators. I basically do too, but I don't see how we can manage to have all translations completed by humans. And even if a few translations are not correct, hopefully a user will get in touch.

If we decide against the translations, we should at least remove the test. If we go for the tranlsations, I'll add the test.

What do you think?

@sdetweil
Copy link
Collaborator

I think it's better to have the translations and take PRs for being wrong. This is such a small space of technical terms, most automated would be right.

I did the same on a product years ago and only had one correction in the Korean translation

@khassel
Copy link
Collaborator

khassel commented May 28, 2025

go for translations

@rejas
Copy link
Collaborator

rejas commented May 29, 2025

I am still sceptic... Could you ask in the forums for people that can check this PR?

@KristjanESPERANTO KristjanESPERANTO changed the title Complete translations [l10n] Complete translations May 29, 2025
@khassel
Copy link
Collaborator

khassel commented May 29, 2025

@rejas as you are active here, can you please read discord and change the permissions needed, thanks.

@khassel
Copy link
Collaborator

khassel commented May 29, 2025

I am still sceptic... Could you ask in the forums for people that can check this PR?

To be honest, I'm not expecting any feedback, so I see it as a waste of time. We can do it and wait for reactions, but the PR should be merged before the next release.

@KristjanESPERANTO
Copy link
Collaborator Author

KristjanESPERANTO commented May 29, 2025

I am still sceptic... Could you ask in the forums for people that can check this PR?

Since we still have some time until the next release, I think it's okay to ask the community for help: https://forum.magicmirror.builders/topic/19709/community-task-check-automated-translations 🙂

@KristjanESPERANTO
Copy link
Collaborator Author

I suggest that if there is no feedback from the forum by Monday we merge this.

@sdetweil
Copy link
Collaborator

The forums have been pretty quiet for a month now. I wouldn't expect much activity on this by Monday

@bugsounet
Copy link
Contributor

@KristjanESPERANTO :

There is nothing to review in fr.json file ?
I don't see it in change files list

@KristjanESPERANTO
Copy link
Collaborator Author

There is nothing to review in fr.json file ?

Yes, that's right. This translation was already complete, so I didn't have to touch it.

KristjanESPERANTO and others added 2 commits June 4, 2025 00:33
Co-authored-by: Magnus <34011212+MagMar94@users.noreply.github.com>
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.

6 participants