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

Require the DAO is enabled for traders #384

Closed
HenrikJannsen opened this issue Oct 24, 2022 · 1 comment
Closed

Require the DAO is enabled for traders #384

HenrikJannsen opened this issue Oct 24, 2022 · 1 comment
Assignees
Labels
a:proposal https://bisq.wiki/Proposals re:protocol was:approved

Comments

@HenrikJannsen
Copy link

To reach the goal to remove the role of the Burningman we propose the distribution of trade fees (#383) and the distribution of the funds from the delayed payout transactions of trades ending up in arbitration (proposal not published yet) to contributors based on their contribution requests. This require that the traders have the DAO data available.
Currently users can choose to deactivate the DAO in the Settings and therefore would not have any DAO data.
If we permit that, we still need a fallback solution for such traders, making the complete removal of the BM role impossible.

This proposal suggests to remove the option to deactivate the DAO in the settings as well as from the program arguments.
Users who have disabled it would switch automatically to the enabled state.
If users would still manage to deactivate it (e.g. by changing the code) their trades would fail.

This change might increase also the chances that users use BSQ for paying the trade fee as a positive side effect.
The downside is that users might need a bit more bandwidth for syncing the DAO data but even if that data sync fails due bad connectivity the app is still usable and as long the user has the data of the last or second last DAO cycle a trade can be completed (details will follow in a future proposal about the delayed payout transactions).

@HenrikJannsen
Copy link
Author

Close as it got merged to master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:proposal https://bisq.wiki/Proposals re:protocol was:approved
Projects
None yet
Development

No branches or pull requests

2 participants