Skip to content
This repository was archived by the owner on Feb 6, 2025. It is now read-only.

Added german translation and added country name support#273

Closed
beckerr-rzht wants to merge 2 commits intoeduvpn:masterfrom
beckerr-rzht:master
Closed

Added german translation and added country name support#273
beckerr-rzht wants to merge 2 commits intoeduvpn:masterfrom
beckerr-rzht:master

Conversation

@beckerr-rzht
Copy link
Copy Markdown

  • Added german translation

  • Added support for country name as translation key.
    Currently in use are "de", "en", "fr" and "nb".
    See:

    wget -qO - https://static.eduvpn.nl/disco/institute_access.json \
      | jq '.instances[].display_name' | awk -F: '/:/{print $1}' | sort -u
    
  • Institution list is sorted after(!) display name translations now.

* Added support for country name as translation key.
  Currently in use are "de", "en", "fr" and "nb".
  See:
  ```
  wget -qO - https://static.eduvpn.nl/disco/institute_access.json \
	| jq '.instances[].display_name' | awk -F: '/:/{print $1}' | sort -u
  ```

* Institution list is sorted after(!) display name translations now.
@ghost
Copy link
Copy Markdown

ghost commented May 14, 2020

Thanks for the PR! Will look at it soon and make a new release!

@ghost
Copy link
Copy Markdown

ghost commented Aug 11, 2020

@dzolnai did you manage to look at this PR?

@dzolnai
Copy link
Copy Markdown
Collaborator

dzolnai commented Aug 31, 2020

Hello @beckerr-rzht,

Sorry for taking this long to respond to you about your PR, but we thought it would be better if we would include the strings after we release version 2.
Otherwise we would end up with a version which has missing strings. Could you update your PR to match the latest version?
Also your edit to SerializerService should be moved to TranslatableString.

@tribut tribut mentioned this pull request May 11, 2023
@dzolnai
Copy link
Copy Markdown
Collaborator

dzolnai commented May 17, 2023

Closing because #386 has fixed this

@dzolnai dzolnai closed this May 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants