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

Remove dao disable toggle and cl options #6391

Conversation

HenrikJannsen
Copy link
Collaborator

@HenrikJannsen HenrikJannsen commented Oct 24, 2022

Implements bisq-network/proposals#384
Base on #6386 starts at e927f13

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.
…r receiving the btc fees.

Add isReducedIssuanceAmountToggle to let the contributor select if the reduced BSQ model is used or not.
`.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>
…m.RECIPIENT_BTC_ADDRESS.getDefaultValue instead if there is no compensation request available.

Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
@ripcurlx ripcurlx added this to the v1.9.7 milestone Oct 25, 2022
Signed-off-by: HenrikJannsen <boilingfrog@gmx.com>
@HenrikJannsen HenrikJannsen deleted the remove_DAO_disable_toggle_and_CL-options branch November 9, 2022 17:55
@ripcurlx ripcurlx removed this from the v1.9.7 milestone Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants