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

Merged
merged 2 commits into from
Nov 7, 2023
Merged

Update de.json #881

merged 2 commits into from
Nov 7, 2023

Conversation

stephanduesterhoeft
Copy link
Contributor

Added a few translations

Added a few translations
@stephanduesterhoeft
Copy link
Contributor Author

Do I have to contribute anything else? I have found other strings to translate and would like to submit them. However, these changes must first be accepted.

de.json Outdated
"label-line-option-description": "suche Begriff in der gleichen Zeile",
"label-section-option-description": "suche Begriff unter der gleichen Überschrift",
"label-property-option-description": "match property",
"label-property-option-description": "Eigenschaften stimmt überein",
Copy link
Collaborator

Choose a reason for hiding this comment

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

"Eigenschaft stimmt überein"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @joethei, could you be a bit more specific? Do you see a mistake in the translation?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The English original is using singular, while your translation is using plural.
(and only partially at that)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Danke. Hatte Tomaten auf den Augen. Habe es entsprechend korrigiert.

@joethei joethei merged commit b00b3d7 into obsidianmd:master Nov 7, 2023
1 check passed
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.

2 participants