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

Updated german translations #468

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Updated german translations #468

merged 1 commit into from
Oct 26, 2023

Conversation

peat80
Copy link
Contributor

@peat80 peat80 commented Oct 25, 2023

Updated german strings

Updated german strings
Copy link
Contributor

@AnonymousWP AnonymousWP left a comment

Choose a reason for hiding this comment

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

Same as here: #467 (comment). Also, do not close this PR like you did previously, but you can apply my suggestions.

Comment on lines +44 to +45
<string name="source_fdroid" translatable="false">F-Droid Offiziell</string>
<string name="source_izzy" translatable="false">F-Droid Izzy</string>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<string name="source_fdroid" translatable="false">F-Droid Offiziell</string>
<string name="source_izzy" translatable="false">F-Droid Izzy</string>
<string name="source_fdroid" translatable="false">F-Droid (Main)</string>
<string name="source_izzy" translatable="false">F-Droid (Izzy)</string>

Again, you shouldn't modify those.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just look at the main strings.xml in the values folder.
Those strings were also changed and have the brackets removed.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, that's fair, but you shouldn't put "Officiziell". Keep it as-is with out the brackets.

Copy link
Owner

Choose a reason for hiding this comment

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

Maybe I should mark these as translatable.

<string name="source_aptoide" translatable="false">Aptoide</string>
<string name="source_github" translatable="false">GitHub</string>
<string name="source_gitlab" translatable="false">GitLab</string>
<string name="source_apkpure" translatable="false">APKPure</string>
Copy link
Owner

Choose a reason for hiding this comment

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

There's no need to add strings that are marked as non traslatable

@rumboalla rumboalla merged commit fb7afab into rumboalla:3.x Oct 26, 2023
1 check passed
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.

3 participants