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

Translations in Transifex #2062

Closed
Valdnet opened this issue Jun 24, 2021 · 10 comments · Fixed by #4274
Closed

Translations in Transifex #2062

Valdnet opened this issue Jun 24, 2021 · 10 comments · Fixed by #4274

Comments

@Valdnet
Copy link
Contributor

Valdnet commented Jun 24, 2021

I noticed that there are a lot of text strings that are not translated in Transifex. The lack of translations results in a slightly underdeveloped NC instance.

@skjnldsv Is it possible to easily add these strings to be translated? This is a priority issue 😉.

@skjnldsv
Copy link
Contributor

I think this is the issue where all translations fail to be extracted from the templates.
I think we could maybe consider extracting the strings from the dev builds instead of source 🤔

@raimund-schluessler
Copy link
Contributor

@Valdnet Is this still the case? If so, could you show the strings that are not translated, please?

@Valdnet
Copy link
Contributor Author

Valdnet commented Jan 5, 2022

@raimund-schluessler There are still many text strings that are not translated in Transifex. You must review all of your code to catch these text strings as they are in different files.

@raimund-schluessler
Copy link
Contributor

@raimund-schluessler There are still many text strings that are not translated in Transifex. You must review all of your code to catch these text strings as they are in different files.

So they are on Transifex already, just not translated? This is nothing we can solve here, in that case the translators on Transifex need to catch up with the strings.

@Valdnet
Copy link
Contributor Author

Valdnet commented Jan 5, 2022

So they are on Transifex already, just not translated?

@raimund-schluessler No. The text strings are not adapted at all for the Transifex bot to scan them and add them to the translation.

@raimund-schluessler
Copy link
Contributor

@raimund-schluessler No. The text strings are not adapted at all for the Transifex bot to scan them and add them to the translation.

Could you then please name the strings for which you noticed it? Saying "somewhere in your code" is not very helpful 🙈

@raimund-schluessler
Copy link
Contributor

raimund-schluessler commented Jan 5, 2022

@Valdnet Some strings I fixed (more or less recently), see 05171d8 So I wonder what else is missing. Please note, these strings to translate are on transifex, but might not be released and propagated to the apps yet.

@Valdnet
Copy link
Contributor Author

Valdnet commented Jan 5, 2022

Some strings I fixed (more or less recently), see 05171d8

👍

Please note, these strings to translate are on transifex, but might not be released and propagated to the apps yet.

Okay, then let's wait for them to be added to Transifex, and then see what it will be like in the instance 😉.

@Valdnet
Copy link
Contributor Author

Valdnet commented Apr 4, 2022

Still no text strings in Transifex with issue: #512.

@susnux
Copy link
Contributor

susnux commented Jun 26, 2023

After #4272 following string are missing:

Open contact menu
Clear search

Got this by just a quick search on the source code:

while read p; do
    grep "\"$p\"" l10n/messages.pot 2>&1 > /dev/null || echo "$p";
done < <(grep -hRPo "(?<=\bt\(')[^']+" src)

False positive because of ":

Actions for item with name "{name}"
Open link to "{resourceName}"
Write message, use "@" to mention someone, use ":" for emoji autocompletion …
Load more "{options}""

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 a pull request may close this issue.

4 participants