-
Notifications
You must be signed in to change notification settings - Fork 492
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
Maximize Privacy CJ Profile Anonscore display fix #9593
Maximize Privacy CJ Profile Anonscore display fix #9593
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.
tACK
WalletWasabi.Fluent/ViewModels/CoinJoinProfiles/CoinJoinProfilesViewModel.cs
Outdated
Show resolved
Hide resolved
WalletWasabi.Fluent/ViewModels/CoinJoinProfiles/CoinJoinProfilesViewModel.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.
cACK.
WalletWasabi.Fluent/ViewModels/CoinJoinProfiles/CoinJoinProfilesViewModel.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
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
@soosr is there a blocker in this PR? |
…e_anonscore_fix Maximize Privacy CJ Profile Anonscore display fix
Fixes this issue #9588 (comment) found by @turbolay
The issue:
When we change the anonscore target of the
Maximize Privacy
CoinJoin profile, we save the anonscore target to thekeymanager
, but we don't update theMaximize Privacy
profile.So the randomly selected anonscore target (between 50-101) will be displayed instead of the user's selection.
Nice video of the issue can also be found here #9588 (comment)