-
Notifications
You must be signed in to change notification settings - Fork 85
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
Translated SPARQL interface to DE #470
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
General comment
I am not a German speaker, therefore, I only performed syntax and format check.
Testing
No expertise for this
Code reviewing
Looks good regarding syntax and format.
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hauschke I have created code suggestions from @matthiasluehr comments. In the case you decide to adopt some of Matthias comments, you can just click on Commit changes.
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
home/src/main/resources/rdf/i18n/de_DE/interface-i18n/firsttime/vitro_UiLabel.ttl
Outdated
Show resolved
Hide resolved
…e/vitro_UiLabel.ttl Co-authored-by: Dragan Ivanovic <[email protected]>
…e/vitro_UiLabel.ttl Co-authored-by: Dragan Ivanovic <[email protected]>
…e/vitro_UiLabel.ttl Co-authored-by: Dragan Ivanovic <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Looks good. Due to lack of a proper german translation I think we could leave "SPARQL Query Builder" as it is.
VIVO GitHub issue: vivo-project/VIVO#3990
What does this pull request do?
Adds labels for German translation of /admin/sparqlquery.
What's new?
Untranslated labels are now translated, some missing hyphens were added, too.
How should this be tested?
A German native speaker should have a look.
Interested parties
@VIVO-project/vivo-committers, German speaking VIVO users
Reviewers' expertise
Reviewers' report template
Please update the following template which should be used by reviewers.
General comment
A reviewer should provide here comments and suggestions for requested changes if any.
Testing
A reviewer should briefly describe here how it was tested
Code reviewing
A reviewer should briefly describe here which part was code reviewed