-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Use environment locale to select right locale and currency #1313
Use environment locale to select right locale and currency #1313
Conversation
Have you tested that with non en_US locales? I don't remember what was the issue but I think there have been an issue with not using en US here. |
Yes, I've tested it with
I don't see any errors in logs. |
Hm, although stuff works for this ticket, I've just realized that language is always overwritten with EN due to this: |
Yes there have been issues with NON English locales. Can you try to deactivate the code in https://github.com/bisq-network/exchange/blob/master/gui/src/main/java/io/bisq/gui/app/BisqAppMain.java#L37 and test with non English locales? |
I did that with German and got German translation from the beginning.
1 lut 2018 22:01 "Manfred Karrer" <[email protected]> napisał(a):
… Yes there have been issues with NON English locales. Can you try to
deactivate the code in https://github.com/bisq-
network/exchange/blob/master/gui/src/main/java/io/bisq/gui/
app/BisqAppMain.java#L37 and test with non English locales?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1313 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAaT2MWrLeE4msHtHJ_r9pfmqnojX5euks5tQiYWgaJpZM4R1lWT>
.
|
As far I remember it was some more subtle issue... damn I should have commented it more in details.... |
Should we merge it then? |
It is missing the removal of the english locale setting at startup. Can you add that? |
212d8d4
to
4b0692c
Compare
Removed setting default language to English at startup. |
Fixes #1204