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

Improve fee handling #5826

Merged
merged 10 commits into from
Nov 18, 2021
Merged

Conversation

chimp1984
Copy link
Contributor

No description provided.

be used in follow-up commits)
Used in case of maker fee validation
Cleanups
That check is done if:
- If we are after activation time
- If filter is available
- If value in filter is > 0

The check compares the paid fee and the fee value from filter
and requires that paid fee is > 70% of filter value.
See comments in code for more background.
@chimp1984 chimp1984 force-pushed the improve-fee-handling branch from 5e940a5 to b00d317 Compare November 14, 2021 14:08
@ripcurlx ripcurlx added this to the v1.8.0 milestone Nov 16, 2021
Copy link
Contributor

@ripcurlx ripcurlx left a comment

Choose a reason for hiding this comment

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

utACK - I only reviewed the code and tested filter creation on Regtest. For Mainnet tests I'll wait for the ACK from @jmacxx before I'll merge this PR.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

ACK

@ghost ghost mentioned this pull request Nov 18, 2021
@ripcurlx ripcurlx merged commit a27ac43 into bisq-network:master Nov 18, 2021
@chimp1984 chimp1984 deleted the improve-fee-handling branch November 21, 2021 19:08
@ghost ghost mentioned this pull request Jan 3, 2022
@ghost ghost mentioned this pull request Apr 27, 2023
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