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

Add option to hide non supported payment methods #5045

Conversation

chimp1984
Copy link
Contributor

Add option in preferences to hide payment methods from combobox in the Offer views which are not part of the users accounts.

Default value is false, so same behaviour as before that PR.
If no payment account is setup then we also show all payment methods.
In that case (no payment account) we disable the toggle as well as show it in false state.

Based on #5042
Starts at commit eddb7cb

currencies as TransferWise supports sending to
most currencies.
…of the users accounts.

Default value is false, so same behaviour as before the change.
If no payment account is setup then we also show all payment methods.
In that case (no payment account) we disable the toggle as well as set it to false.
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

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

ACK

Took me a while to figure it was just the combo box this filtered. Should've read the PR description more thoroughly.

@sqrrm sqrrm merged commit 7bc9c10 into bisq-network:master Jan 4, 2021
@chimp1984 chimp1984 deleted the add-option-to-hide-non-supported-payment-methods branch January 4, 2021 16:12
@ripcurlx ripcurlx added this to the v1.5.5 milestone Jan 7, 2021
@Conza88
Copy link

Conza88 commented Feb 2, 2021

Kind of confused what this changes exactly? Photo may help?

The toggle already removes (hides) non supported payment methods?

image

@ripcurlx
Copy link
Contributor

ripcurlx commented Feb 3, 2021

Kind of confused what this changes exactly? Photo may help?

The toggle already removes (hides) non supported payment methods?

image

Using that toggle hides all greyed out offers you can't take as you don't have a corresponding payment account to do so.

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