-
Notifications
You must be signed in to change notification settings - Fork 40
Update de_DE translation #112
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
Conversation
Signed-off-by: Sören Wünsch <soeren.wrede@parshipgroup.com>
|
Thank you! I'm keeping this open, as there are likely going to be changes in strings for 0.4.0, so updating translations closer to the next release is probably needed. |
Signed-off-by: Sören Wünsch <soerenwrede@gmail.com>
|
@Soean @dertuxmalwieder - Is this ready for the current version? |
Signed-off-by: Sören Wünsch <soeren.wrede@parshipgroup.com>
|
@Ipstenu Yes, I updated and translated it. |
|
LGTM too. ;-) |
|
Thank you both! You rule! |
|
So we only really need the .po file as the rest are created via a GitHub Action here. It also seems that there are conflicts with the .po in the PR. I know it's a big ask, but we can merge as soon as this is fixed. |
# Conflicts: # languages/fair-de_DE.l10n.php # languages/fair-de_DE.mo # languages/fair-de_DE.po
|
Thanks for your feedback. I updated the po file and merged the main branch. |
|
@Soean Thanks a lot for the updates! ❤️ The last three commits aren't signed off so DCO is failing. You can either:
|
Signed-off-by: Sören Wünsch <soeren.wrede@parshipgroup.com>
Signed-off-by: Sören Wünsch <soeren.wrede@parshipgroup.com>
Signed-off-by: Sören Wünsch <soeren.wrede@parshipgroup.com>
|
@costdev You are right, I fixed it |
No description provided.