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

[Autocoinjoin] Freeze utxo from coinjoin #7487

Closed
MaxHillebrand opened this issue Mar 8, 2022 · 14 comments
Closed

[Autocoinjoin] Freeze utxo from coinjoin #7487

MaxHillebrand opened this issue Mar 8, 2022 · 14 comments
Assignees
Milestone

Comments

@MaxHillebrand
Copy link
Collaborator

Currently all utxos are eligable to be registerd for coinjoin. However, there are many good reasons why some specific utxo does not need to be coinjoined.

For example, a trader withdrawls from an exchange in evening, knowing that he will deposit again in the morning. He is sending to the same entity, and thus there is no need to pay for blockspace to get privacy.

I see two ways of implementing this:

  1. In the coin list, have a button or right click menu with Freeze from coinjoin. The coin will only be registered if the user clicks Unfreeze from coinjoin. However, the coin should still be considered for coin selection for single user payments.

  2. In the receive tab when generating a new address, the user can click on a checkbox with Don't coinjoin this address. This would be more direct in the default UX and would be seen on every receive dialog. Since we should probalby not encourage this opt-out feature by default, I would prefer the first approach.

@MaxHillebrand
Copy link
Collaborator Author

this coinjoin icon could be clickable so that it toggles to strike through, indicating that this coin is frozen. when icon is clicked again, the strike-through disappears and the coin will be registered again.

screenshot_20220308_100721

@stale
Copy link

stale bot commented Jun 19, 2022

This has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@soosr
Copy link
Collaborator

soosr commented Jun 27, 2022

cACK

1 similar comment
@BTCparadigm
Copy link
Collaborator

cACK

@tricopol
Copy link

This is an absolute must for me. I'd love to hear what was the rationale behind releasing 2.x without this feature.

@tricopol
Copy link

tricopol commented Oct 28, 2022

  1. In the coin list, have a button or right click menu with Freeze from coinjoin. The coin will only be registered if the user clicks Unfreeze from coinjoin. However, the coin should still be considered for coin selection for single user payments.

Please make sure you can select multiple (or all) coins (like you do Ctrl-A or shift-select in Windows Explorer) in a single pass instead of having to click each coin individually. If i have 100 coins and want to coinjoin 5 of them, freezing 95 one-by-one will get tedious..

@molnard
Copy link
Collaborator

molnard commented Jan 30, 2023

Business logic is done but the UI is unusable - this is as planned. They do not know if the coin is frozen or not, nor how to freeze a coin. The VDG team will take over from this point. Pinging @nopara74

@Pule08 Pule08 assigned ghost and unassigned Szpoti Jan 30, 2023
@Pule08 Pule08 removed their assignment Feb 13, 2023
@Pule08 Pule08 unassigned ghost Mar 13, 2023
@Szpoti
Copy link
Collaborator

Szpoti commented Mar 17, 2023

@zkSNACKs/visual-design-group What's up with this?

@soosr
Copy link
Collaborator

soosr commented Mar 20, 2023

@zkSNACKs/visual-design-group What's up with this?

Hiring a UX designer is in progress, we will wait with this issue (and some others) to see what are their thoughts about the possible solution.

@lukasdll
Copy link

There is already a checkbox and an "action menu" at the bottom with "send selected coins".

Wouldn't it be more intuitive for the user to add the actions "exclude from CoinJoin" and "include in CoinJoin" at the same location, and those actions be applied on the selected coins?

Image

The multi-selection idea, via a combination of keys CTRL+SHIFT, would then improve the inclusion, exclusion but also the submission (via "send selected coins").

We could eventually make buttons out of it, instead of underlined texts.

Image

For the icon, I would like to suggest using the "Ready to CoinJoin" / "CoinJoining" / "Can't participate in CoinJoin until", to include another symbol indicating that the coin has been excluded from CoinJoins manually, as it seems a better place from a "semantic" point of view than the "V" location, that instead seems to indicate whether the coin is an UTXO from a confirmed transaction.

Perhaps we could add a grey colored "Stop hand" to indicate the exclusion of the coin.

Image

An idea of icon:

Image

Image

@soosr
Copy link
Collaborator

soosr commented Sep 18, 2023

@lukasdll

We should rather remove features from the wallet coins dialog. Initially, it was introduced only for developers to help with the development.
Introducing the "Send whole coin" feature was rushed back then, and this dialog was the only place it could be added.
A feature like that or Freeze coins should not only be accessible via a magic key combo and they should be placed where it makes sense.
Send whole coins should be put somewhere around Send, while the Freeze coins should be put somewhere around Coinjoin player.
This topic will be discussed soon among the UX team.

@MaxHillebrand MaxHillebrand added this to the v2.0.6 milestone Dec 25, 2023
@soosr soosr modified the milestones: v2.0.6, v2.0.7 Jan 15, 2024
@soosr soosr self-assigned this Jan 17, 2024
@soosr
Copy link
Collaborator

soosr commented Feb 2, 2024

Reached an agreement in terms of UX and UI designs.

  • Replace the LED indicator with a ... menu in MusicBox
    • (this will allow us to add Coinjoin related stuff there)
  • Add a Exclude Coins options, which pop a dialog with the list of coins.

The team will start to work on it after the UI Refreshment.

@soosr soosr removed the UX label Feb 2, 2024
@soosr soosr removed their assignment Mar 18, 2024
@nostitos
Copy link

Another very common reason to not want to automix all new deposits is that people might want to save on fees and complexity of having two wallets and paying for extra transaction when they want to mix it.

Unless the user is a coin selection expert, That extra transaction is most likely consolidating coins costing a lot in lost privacy.

@soosr soosr modified the milestones: v2.0.7, v2.0.8 Mar 25, 2024
@soosr soosr closed this as not planned Won't fix, can't repro, duplicate, stale May 2, 2024
@soosr soosr reopened this May 6, 2024
@MarnixCroes
Copy link
Collaborator

closing, as #12893 was merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests