Skip to content

Conversation

@pablotheissen
Copy link

I went through the German translation and did a couple of fixes:

  • More natural (and consistent) language
  • Fixes some language errors
  • Consistent use of en-dashes

More natural language
Fixes some language errors

Signed-off-by: Pablo <[email protected]>
@PranshulGG
Copy link
Owner

@pablotheissen Thanks for the fixes!
I can’t merge this PR though, the German translation is managed via Crowdin and would get overwritten.
Could you apply these changes directly on Crowdin instead? That’d be perfect.

@pablotheissen
Copy link
Author

Absolutely! I'll also have a look at the other not yet approved strings of the German translation.

@pablotheissen
Copy link
Author

@PranshulGG some strings are missing on CrowdIn, e.g., high_forecast_uv, which is present in source.json but missing on the UI of CrowdIn.

@PranshulGG
Copy link
Owner

@pablotheissen Should be fixed!

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.

2 participants