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
[VDG] PrivacySuggestions: add warning for spending coinjoining coins #11055
Conversation
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.
The warning is good, but the user should rarely see it, such selection shouldn't be done anyhow.
cACK. I am not sure if it needs to be a warning (Info instead?). Are there any privacy concerns about it? |
@soosr I used the warning as using coinjoining coins might cause issues, not necessarily for privacy pov. I can change it to info instead |
Info makes more sense, as they will still get the green shield in this case. |
WalletWasabi.Fluent/Models/Transactions/PrivacySuggestionsModel.cs
Outdated
Show resolved
Hide resolved
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.
tACK
As spending coinjoining coins causes issues for both client and backend, I think this warning makes a lot of sense.
Issue: similar to #11013, the suggestion doesn't get updated. So user can see the warning while coinjoin stopped in the meantime.