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

Use environment locale to select right locale and currency #1313

Conversation

blabno
Copy link

@blabno blabno commented Feb 1, 2018

Fixes #1204

@ManfredKarrer
Copy link
Contributor

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.

@blabno
Copy link
Author

blabno commented Feb 1, 2018

Yes, I've tested it with en_US. It selects:

  • MainView currency price dropbox: BTC/USD
  • Settings->Base currency: BTC
  • Settings->Language: English
  • Settings->Country: United States
  • Settings->Preferred Currency: USD

I don't see any errors in logs.

@blabno
Copy link
Author

blabno commented Feb 1, 2018

Hm, although stuff works for this ticket, I've just realized that language is always overwritten with EN due to this:
https://github.com/bisq-network/exchange/blob/master/gui/src/main/java/io/bisq/gui/app/BisqAppMain.java#L37

@ManfredKarrer
Copy link
Contributor

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?

@blabno
Copy link
Author

blabno commented Feb 1, 2018 via email

@ManfredKarrer
Copy link
Contributor

As far I remember it was some more subtle issue... damn I should have commented it more in details....
But we can change it, the issue will pop up during dev testing or at least at release testing. The next release will require a profound test cycle anyway.

@blabno
Copy link
Author

blabno commented Feb 3, 2018

Should we merge it then?

@ManfredKarrer
Copy link
Contributor

It is missing the removal of the english locale setting at startup. Can you add that?

@blabno blabno force-pushed the feature/1204-use-environment-locale-to-select-right-locale-and-currency branch from 212d8d4 to 4b0692c Compare February 5, 2018 12:03
@blabno blabno requested a review from ManfredKarrer as a code owner February 5, 2018 12:03
@blabno
Copy link
Author

blabno commented Feb 5, 2018

Removed setting default language to English at startup.

@ripcurlx ripcurlx added the in:gui label Feb 6, 2018
@ManfredKarrer ManfredKarrer merged commit 224a91b into bisq-network:master Feb 13, 2018
@blabno blabno deleted the feature/1204-use-environment-locale-to-select-right-locale-and-currency branch April 6, 2018 09:18
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.

3 participants