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

Minor wording correction to da.xliff #180

Merged
merged 2 commits into from
Feb 15, 2022
Merged

Conversation

JacobSchriver
Copy link
Contributor

'Activate for Duration' shows in english in the preferences window. I thought I missed a line of translation - but actually it was translated. Now also corrected in wording.

'Activate for Duration' shows in english in the preferences window. I thought I missed a line of translation - but actually it was translated. Now also corrected in wording.
@newmarcel
Copy link
Owner

Thanks for highlighting the problem! To test it out I applied this patch, but the problem remains.

I tracked down the issue to:
da.xliff Line 167, there is the <source>Activation Duration</source> entry, but it lacks a translated target line: <target>…</target>.

This should resolve the issue. Can you also make these changes? Then I will merge your pull request. 👍

@JacobSchriver
Copy link
Contributor Author

Thanks for highlighting the problem! To test it out I applied this patch, but the problem remains.

I tracked down the issue to: da.xliff Line 167, there is the <source>Activation Duration</source> entry, but it lacks a translated target line: <target>…</target>.

This should resolve the issue. Can you also make these changes? Then I will merge your pull request. 👍

Apparently my eyes missed that one - thanks for highlighting it 👍🇩🇰

@newmarcel newmarcel merged commit 95784e8 into newmarcel:main Feb 15, 2022
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.

None yet

2 participants