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 language #713

Merged
merged 2 commits into from
Mar 17, 2021
Merged

update german language #713

merged 2 commits into from
Mar 17, 2021

Conversation

4rchim3d3s
Copy link
Contributor

Did you run make format && make check?
No

Changes:

  • update german language and copied latest en.json to de_base.json

Questions:

  1. What should be the german word for "Visor"?
    -> "der Visor" ?

  2. And what's the plural then?
    -> I used "die Visor"

  3. What's prefered: "Update" or "Aktualisierung" ?

@jdknives
Copy link
Member

@4rchim3d3s I would stick with

  • Aktualisierung
  • Visor/Visor

Thanks a lot for this PR. Will check and merge after v0.4.0 is out.

Copy link
Member

@jdknives jdknives left a comment

Choose a reason for hiding this comment

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

Good job overall. Thanks. Just a few comments.

static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
"error-load": "Beim Aktualisieren der Visor-Liste ist ein Fehler aufgetreten.",
"empty": "Es ist kein Visor zu diesem Hypervisor verbunden.",
"empty-with-filter": "Kein Visor erfüllt die gewählten Filterkriterien",
Copy link
Member

Choose a reason for hiding this comment

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

Punctuation at the end of the sentence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Edited. Should we edit punctuation on all sentences? I think there are a few without at the moment.

static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
"copied-pk-info": "Öffentlicher Schlüssel wurde kopiert",
"copy-pk-error": "Beim Kopieren des öffentlichen Schlüssels ist ein Problem aufgetreten.",
"no-elements": "Derzeit können keine Elemente angezeigt werden. Bitte später versuchen.",
"no-elements-for-filters": "Keine Elemente, welche die Filterkriterien erfüllen",
Copy link
Member

Choose a reason for hiding this comment

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

May need punctuation. Not sure what would be correct since this is not really a sentence, but feels like one. 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, maybe also Keine Elemente vorhanden, ... ?

static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
static/skywire-manager-src/src/assets/i18n/de.json Outdated Show resolved Hide resolved
Copy link
Contributor

@Senyoret1 Senyoret1 left a comment

Choose a reason for hiding this comment

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

I can not comment about the translation, but the changes seems to be fine from a technical point of view 👍

@jdknives
Copy link
Member

Good job. Will merge. @4rchim3d3s @Senyoret1

@jdknives jdknives merged commit 6ef8046 into skycoin:develop Mar 17, 2021
@4rchim3d3s 4rchim3d3s deleted the german-language-update branch March 26, 2023 19:37
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.

3 participants