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

Create offer using the same tx fee specified at funding #4345

Merged
merged 1 commit into from Jul 3, 2020
Merged

Create offer using the same tx fee specified at funding #4345

merged 1 commit into from Jul 3, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jun 29, 2020

This fixes an issue whereby updates from the fee API were causing Insufficient Funds exception. Since the wallet is funded with a specific amount (deposit+trade fee+txFee), that same amount has to be used when the user confirms offer creation (which could be some time later).

Fixes #4227
Fixes #4278
Fixes #4336
Fixes #4327

This fixes an issue whereby updates from the fee API were causing
Insufficient Funds exception. Since the wallet is funded with a
specific amount (deposit+trade fee+txFee), that same amount has
to be used when the user confirms offer creation (which could be
some time later).

Fixes #4227
Fixes #4278
Fixes #4336
Fixes #4327
Copy link
Member

@sqrrm sqrrm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK

@sqrrm sqrrm merged commit ac2f37f into bisq-network:master Jul 3, 2020
@ripcurlx ripcurlx added this to the v1.3.6 milestone Jul 6, 2020
@ripcurlx ripcurlx changed the title Create order using the same tx fee specified at funding Create offer using the same tx fee specified at funding Jul 7, 2020
@ghost ghost mentioned this pull request Jul 12, 2020
@ghost ghost mentioned this pull request Aug 3, 2020
@ghost ghost deleted the fix_offer_fee_changed branch December 3, 2020 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants