-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
Max total fee #1488
Max total fee #1488
Conversation
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.
It looks like you mean "max absolute fee" rather than "max base fee."
I think what we want is "max fee relative" and "max fee absolute" but I'm not sure that's pleb friendly.
Maybe "max fee rate" and "max fee total" |
I think we also want the subtitle to say "we'll use whichever is higher" given the withdrawn amount. |
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.
This probably needs to account for #1499?
Description
This adds
max base fee
as a new autowithdrawal setting. This can be used to make small withdrawals possible without allowing a higher max fee rate for big withdrawals.This setting is set to 0 for users which have autowithdrawals enabled to keep existing fee settings and not suddenly allow higher fees without their interaction.
close #1383 based on #1487
Screenshots
Additional Context
max fee rate
,max base fee
and pathfinding?Checklist
Are your changes backwards compatible? Please answer below:
yes, new db column added but is nullable
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
6
. I checked that max fee is correctly calculated during autowithdrawal.For frontend changes: Tested on mobile? Please answer below:
yes
Did you introduce any new environment variables? If so, call them out explicitly here:
no