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 #2282

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Amr-Ibra
Copy link
Contributor

@Amr-Ibra Amr-Ibra commented Sep 6, 2024

Note:

Poedit can delete many “unused translations”. The question is: is Poedit right? Are there really so many unused translations to delete? Have the original strings really got removed from the source code?

I haven't deleted any “unused translations” yet. But I can update this PR to delete them after confirmation from the developers that the original strings got removed from the source code.

@3v1n0
Copy link
Collaborator

3v1n0 commented Sep 16, 2024

Mh, I feel something is wrong since some of the deleted translations still seems to be used here.

@Amr-Ibra
Copy link
Contributor Author

make mergepo is the one to blame.

@xalt7x
Copy link
Contributor

xalt7x commented Dec 11, 2024

@Amr-Ibra
Somehow I've got incomplete file as well...
You can just rm po/dashtodock.pot, then make potfile

Copy link
Collaborator

@sergio-costas sergio-costas left a comment

Choose a reason for hiding this comment

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

A little change requested.

po/de.po Show resolved Hide resolved
@sergio-costas
Copy link
Collaborator

@3v1n0 @Amr-Ibra I had the same problem recently when working with DevedeNG: I found that intltool-update doesn't recognize the .ui files anymore, although before it did. That's why there are so many sentences marked as deleted translations.

@sergio-costas
Copy link
Collaborator

I was able to fix it by using msgmerge, but that will require a change in the Makefile... I'll see what I can do.

@sergio-costas
Copy link
Collaborator

Oh... the makefile doesn't use intltool-update, but msgmerge... forget what I said.

@Amr-Ibra
Copy link
Contributor Author

Somehow I've got incomplete file as well... You can just rm po/dashtodock.pot, then make potfile

I did that, and I think it fixed the problem. There aren't as many deleted translations as there were before I pushed my last change. I think it's safe to merge this PR now.

Copy link
Collaborator

@sergio-costas sergio-costas left a comment

Choose a reason for hiding this comment

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

Looks fine to me, but somebody else should review it too.

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.

4 participants