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

Initial german translation #179

Merged
merged 1 commit into from
Dec 26, 2020
Merged

Initial german translation #179

merged 1 commit into from
Dec 26, 2020

Conversation

tweimer
Copy link
Contributor

@tweimer tweimer commented Dec 12, 2020

This is still work in progress, Many parts untranslated.

See #164

@tweimer
Copy link
Contributor Author

tweimer commented Dec 12, 2020

@anxdpanic What is the difference between "Clear data cache" and "Delete data cache"? To me, both means the same thing.
A bit more explanation in the blue text would be helpful.

@anxdpanic
Copy link
Owner

anxdpanic commented Dec 12, 2020

@tweimer
Clear data cache: Removes/Clears the entries from the data cache database, leaving an empty database table
Delete data cache: Deletes the data cache database file, this would be used on a corrupt database usually

@tweimer
Copy link
Contributor Author

tweimer commented Dec 12, 2020

@anxdpanic Thanks! Could you add that to the blue help text within the addon? I find that a bit hard to understand.

@anxdpanic
Copy link
Owner

Created an issue so I get to it soon
#180

@anxdpanic
Copy link
Owner

I updated the strings, in #181 you might have to manually align these msgids in your PR.

@anxdpanic anxdpanic self-requested a review December 13, 2020 21:48
@anxdpanic anxdpanic added the l10n Localization label Dec 13, 2020
@tweimer tweimer marked this pull request as ready for review December 26, 2020 20:46
@tweimer
Copy link
Contributor Author

tweimer commented Dec 26, 2020

@anxdpanic Ready to merge!

Copy link
Owner

@anxdpanic anxdpanic left a comment

Choose a reason for hiding this comment

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

Huge thank you for this 👍

@anxdpanic anxdpanic merged commit facf195 into anxdpanic:master Dec 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l10n Localization
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants