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

Update German translation #18261

Closed

Conversation

da-phil
Copy link
Contributor

@da-phil da-phil commented Jan 21, 2025

Update DE translation after new translatable strings were recently added.

@MStraeten
Copy link
Collaborator

MStraeten commented Jan 22, 2025

whats the base for this translation update?
there wasn't a darktable.pot file update since latest merged german translation update

@da-phil
Copy link
Contributor Author

da-phil commented Jan 22, 2025

whats the base for this translation update? there wasn't a darktable.pot file update since latest merged german translation update

One of @TurboGit and a PR of mine added new translations.

Sorry, it's the first time I'm contributing to the translations, hence I'm not entirely sure how it works.
So you're saying that the darktable.pot file needs to be updated first, before individual translations are updated?

@MStraeten
Copy link
Collaborator

at least thats the usual way

@TurboGit
Copy link
Member

So you're saying that the darktable.pot file needs to be updated first, before individual translations are updated?

Yes, I do that regularly except that we were working on stabilizing darktable 5.0.x so less time for that. Anyway, you've probably seen that the .pot file has been updated yesterday and merging window on master is open now.

@MStraeten
Copy link
Collaborator

compared it to a de.po file updated with recently updated darktable.pot file.
this commit doesn't contain any new translations, so where's the benefit?

@TurboGit
Copy link
Member

Let me close this then.

@TurboGit TurboGit closed this Jan 22, 2025
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