Skip to content

Revise: change method to calculate translation statistics#3877

Closed
SlySven wants to merge 2 commits intoMudlet:developmentfrom
SlySven:Revise_changeMethodToCalculateTranslationStats
Closed

Revise: change method to calculate translation statistics#3877
SlySven wants to merge 2 commits intoMudlet:developmentfrom
SlySven:Revise_changeMethodToCalculateTranslationStats

Conversation

@SlySven
Copy link
Copy Markdown
Member

@SlySven SlySven commented Jun 6, 2020

Recently some change at CrowdIn has resulted in the translation files all having text assigned to the translation (output) element in the mudlet_xx_YY.ts files - even if it is the original source text. This means that lrelease will not longer generate extra lines warning about untranslated source texts and we have to switch to counting the texts that are "finished" instead.

Just in case things change again, a warning message will be printed out should those messages reoccur.

This is urgent as it needs to be in for the next release which is imminent - it closes #3779 .

Signed-off-by: Stephen Lyons slysven@virginmedia.com

Recently some change at CrowdIn has resulted in the translation files
all having text assigned to the translation (output) element in the
`mudlet_xx_YY.ts` files - even if it is the original source text. This
means that `lrelease` will not longer generate extra lines warning about
untranslated source texts and we have to switch to counting the texts
that are "finished" instead.

Just in case things change again, a warning message will be printed out
should those messages reoccur.

Signed-off-by: Stephen Lyons <slysven@virginmedia.com>
@SlySven SlySven added the high label Jun 6, 2020
@SlySven SlySven added this to the 4.9.0 milestone Jun 6, 2020
@add-deployment-links
Copy link
Copy Markdown

add-deployment-links bot commented Jun 6, 2020

Hey there! Thanks for helping Mudlet improve. 🌟

Test versions

You can directly test the changes here:

No need to install anything - just unzip and run.
Let us know if it works well, and if it doesn't, please give details.

Copy link
Copy Markdown
Contributor

@Kebap Kebap left a comment

Choose a reason for hiding this comment

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

Superseded by #3878

@Kebap Kebap removed this from the 4.9.0 milestone Jun 8, 2020
@Kebap Kebap removed the high label Jun 8, 2020
@Kebap Kebap closed this Jun 8, 2020
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.

Translation stats are generated incorrectly

3 participants