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 de.json #873

Merged
merged 3 commits into from Jul 14, 2022
Merged

Update de.json #873

merged 3 commits into from Jul 14, 2022

Conversation

ChrisTG742
Copy link
Contributor

See #851

i18n/de.json Outdated
"settings.smtp.testConnection": "Test connection",
"settings.smtp.toEmail": "To e-mail",
"settings.smtp.testConnection": "Verbindung testen",
"settings.smtp.toEmail": "An E-mail",
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't this more like "Empänger E-Mail" than "An E-Mail". That sounds strange. Do you use it like that in normal every-day speech?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The question is the context this is used in. If it is ment like "recipient", then I'm with you that should translate to "Empfänger". But I thought is is ment as in the sentence "should be sent to", then "An" fits better here.
Maybe @knadh could shed some light in here?
TIA

Copy link
Owner

Choose a reason for hiding this comment

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

image

This is how it looks on the UI. Sending test message from email -> to e-mail.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Okay, then @an0nfunc is right, I'll fix that.

@knadh knadh merged commit 650e23e into knadh:master Jul 14, 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

3 participants