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

Added nl-NL translations. #132

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

rbrouwer
Copy link

No description provided.

@Melvin-Abraham
Copy link
Owner

@rbrouwer Thank you for the contribution. However, Google Assistant SDK doesn't seem to support this language code as of now. Thus, if you try asking a query to the assistant (after fixing the JSON syntax error) with language set to this one, it will throw the following error (in DevTools):

Invalid language code error

NOTE: For future reference (in case Google adds support for this language in Assistant SDK):
https://developers.google.com/assistant/sdk/reference/rpc/languages

@rbrouwer
Copy link
Author

rbrouwer commented Feb 14, 2021

I will talk Dutch to it anyway. Google is smart enough to understand that anyway (really!) and it will also talk back in Dutch. Doing that will however change the app in part Dutch and part the setup language. That's why thought adding those translations would help.
afbeelding

Possibly the "GUI" language and "SDK" language should be kept separate? That could possibly be done by adding an SDK language setting in this json structure.

I've fixed this json issue just in case.

@rbrouwer
Copy link
Author

I've separated "GUI language" and "Assistant/SDK language", allowing people to have those few translated lines in 1 language, while "communicating with Assistant" in another.

In case of nl-NL that actually works as you would expect it to work; there might be more languages which will act the same way, but that's for other developers to try out and add.

@Melvin-Abraham
Copy link
Owner

Melvin-Abraham commented Feb 15, 2021

@rbrouwer That is doable but currently I will leave this PR on hold and get back to it once a proper way of internationalization for the entire UI is figured out.

@yahoo1999
Copy link

NOTE: For future reference (in case Google adds support for this language in Assistant SDK):
https://developers.google.com/assistant/sdk/reference/rpc/languages

@Melvin-Abraham I feel like this list maybe somewhat incomplete. I am 100% sure that the Google Assistant supports Polish officially. (There's even official homepage for it). Yet it's not on the list.

@rbrouwer
Copy link
Author

rbrouwer commented Mar 5, 2021

@yahoo1999 , Polish is one of those languages like Dutch. It's not "supported" by the SDK/in that list, but when the language is enabled in your account and you just speak that language to Google Assistant, it will just respond in that same language.

I believe there are a few more languages like that! Norwegian also exists for that page and is not in the SDK list.

Melvin-Abraham however does not know how he really wants to handle I18n, so I guess we just have to wait until that is decided.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants