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

Update default fiat and crypto currencies #3179

Conversation

devinbileck
Copy link
Member

Within the settings, the default list of displayed national currencies and altcoins is outdated, in terms of top traded, and most are unnecessary to be displayed for most users and a nuisance to have to remove (which I am sure is the first thing most users do).

I have updated it so that the preferred currency and only displayed national currency is chosen based on the users locale/county code, and the only altcoin displayed by default is BSQ. If the user wants anything else it is up to them to add it.

Before:
image

After:
image

All altcoins have very low trading activity, other than Monero.
I figured we should remove them all anyways, and make the
user add those which they are interested in seeing. The only
one they will see by default being BSQ.
@ripcurlx
Copy link
Contributor

ripcurlx commented Sep 2, 2019

@devinbileck You've to fix the PreferencesTest. If you have time to fix it right now, I'll wait for the codefreeze until you've updated this PR.

Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NACK - Please see my comment

@ripcurlx
Copy link
Contributor

ripcurlx commented Sep 2, 2019

@devin I just looked at the code part. I think this change needs to have some discussion as we are using it right now to push a couple of currencies. @ManfredKarrer @sqrrm @m52go might want to join this discussion as well.

@ripcurlx
Copy link
Contributor

ripcurlx commented Sep 2, 2019

to push a couple of currencies

On the fiat side I agree that it might not make sense to have all other currencies there. On the Altcoin side we have to discuss which ones we want to keep to make it easier for new users to find their coin immediately. To select your Fiat and/or Altcoin market you want to trade in should be part of the future onboarding process. In that case we can remove all of them afterwards.

@devinbileck
Copy link
Member Author

Ah, good points. I will wait to receive feedback before proceeding.

@chimp1984
Copy link
Contributor

I don't think it is good to reduce the fiat currencies to the users default currency, as some users trade on several currencies and might be interested to look up other markets. With one currency it might give the impression Bisq is supporting only that currency for newbies.

@chimp1984
Copy link
Contributor

Also on altcoins I think we should keep a list of "recommended" coins. It is not only about trade volume but about coins which we think makes sense (e.g. Bitcoin Rhodium had high volume but is irrelevant long term, stablecoins will become important as alternative to fiat so we should promote those to make them more visible, as well as privacy coins like Grin - once added again).
Also XMR is 99% volume so that is for all reasons a must have.

Copy link
Contributor

@chimp1984 chimp1984 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Concept NACK

@battleofwizards
Copy link
Contributor

I confirm the multi fiat use case. On top of that I never traded against my locale-implied currency.

@devinbileck devinbileck closed this Sep 3, 2019
@devinbileck devinbileck deleted the update-default-fiat-and-crypto-currencies branch September 3, 2019 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants