-
-
Notifications
You must be signed in to change notification settings - Fork 51
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
128 - Translations and messages improvements #234
Conversation
Hey @abbyad , would you mind to review? Some of the changes you suggested were applied before, and I have fixed others here. I also added more French translations, and Spanish because although we don't use them, it's easy for me. |
47d25de
to
6092f78
Compare
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! Made some minor edits, and also removed CHT-Core throughout since it isn't meant to be used as the branded tool. We should stick to "CHT server" or "CHT instance" as needed in the context.
On a related note, do we even need to say CHT, or would just something like "the server is not ready"? Keeping it generic might help if these messages are coming up on a rebranded app (eg partner branded instead of CHT branded).
Totally agree, I have applied all the changes suggested, and checked there are no remaining CHT-Core references. Please take a look again. |
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
Co-authored-by: Marc Abbyad <[email protected]>
415b5d6
to
8fb1e8c
Compare
It seems as though we agree that the specific mentions to the CHT in these messages should be removed, is that already part of this PR? (I still see "CHT server is not ready, please try again soon") |
So so, I have changed all references of CHT-Core to just CHT or "server" depending of the context, @abbyad do you suggest to also replace "CHT server" for just "Server" ? |
Are these messages seen only when using the unbranded app? If so then CHT may be fine. But if a message like "CHT server is not ready, please try again soon" ( |
Yes @abbyad , all the translations that refers to CHT here are for the Unbranded flavor, and used when you try to setup the CHT instance URL, so I think we are fine, so if it is OK for you please give it the ✔️ . |
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.
Nice work!
Add missed translations for some buttons and messages, and fixed others. Co-authored-by: Marc Abbyad <[email protected]>
Issue: #128