-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Distribute BTC trade fees to contributors #6386
Distribute BTC trade fees to contributors #6386
Conversation
Thanks for opening this pull request! |
If set to true we do not fail validation in case there is no input text
…aMap. We cannot add new instance fields as that would break DAO consensus (hash for monitor). By using the extraMap there is no change in the DAO data. Not updated nodes simply ignore the usage of those data.
2803e0f
to
f567e30
Compare
…r receiving the btc fees. Add isReducedIssuanceAmountToggle to let the contributor select if the reduced BSQ model is used or not.
f567e30
to
13397f9
Compare
Not ready for merge before bisq-network/proposals#383 is not confirmed by the community. |
`.run` directory can be used to share run configurations. Might be useful to check in and share some default configs. If so we need to remove that gitignore entry again. Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
Use first input instead of second from the compensation request tx for getting the receiver address. We cannot use a BTC input as we don't have the spending tx. For BSQ we have all transactions and can get the address from the spending output. Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
…used Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
… due to existing static import 'bisq.desktop.util.FormBuilder.*' Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
There are a few 100 000 txOutputs and iterating them inside the iteration (about 700 issuances) took more than 20 seconds. Now we use a lookup map and it takes about 130 ms. Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
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.
I haven't tested these changes yet but they look correct.
Implementing bisq-network/proposals#383