jam icon indicating copy to clipboard operation
jam copied to clipboard

feat: adapt `absurd_fee_per_kb` setting

Open theborakompanioni opened this issue 1 year ago • 13 comments

Currently, absurd_fee_per_kb is hardcoded in Jam and used when validating the fee settings. absurd_fee_per_kb should be customizable by the user and it should be taken into account when validating fee values.

e.g. a user setting tx fee to 349 sats/vbyte with 20% randomization factor will likely have this tx failing without being properly informed exactly why (tx fee > absurd fee).

theborakompanioni avatar Jan 17 '24 12:01 theborakompanioni

Jam could warn user and if he still agrees dynamically raise absurd_fee_per_kb setting.

kristapsk avatar Jan 17 '24 12:01 kristapsk

Jam could warn user and if he still agrees dynamically raise absurd_fee_per_kb setting.

Yep. Generally "fees and estimates" and how it is best possible to display information to a user so he has even the slightest ideas of what is going on is rather tricky with the API at the moment. It is suboptimal when all possible settings are displayed. The UI should remain simple, but still somehow allow you to adapt most options. Thank God I'm not a designer and @editwentyone has to deal with that and will find a proper solution :crossed_fingers:

theborakompanioni avatar Jan 17 '24 14:01 theborakompanioni

Here's my proposal for sendpayment.py - https://github.com/JoinMarket-Org/joinmarket-clientserver/pull/1636.

kristapsk avatar Jan 17 '24 14:01 kristapsk

Here's my proposal for sendpayment.py - JoinMarket-Org/joinmarket-clientserver#1636.

I appreciate the hint @kristapsk :pray: This could be a viable solution in the web ui too. This, or prompting the user to adapt the absurd_fee_per_kb before the transaction.

theborakompanioni avatar Jan 17 '24 14:01 theborakompanioni

I was briefly thinking maybe RPC should also adjust absurd fee if manual fee is specified above it, but decided that probably it's not a good idea, better to require clients to use configset to override it manually, it's safer.

kristapsk avatar Jan 17 '24 14:01 kristapsk

hi, I'd like to work on this

r3yc0n1c avatar Mar 09 '25 23:03 r3yc0n1c

hi, I'd like to work on this

Nice! This would be perfect in the Fee settings modal šŸ™ I assigned you to the task šŸ’Ŗ

theborakompanioni avatar Mar 10 '25 21:03 theborakompanioni

On it šŸ™Œ

r3yc0n1c avatar Mar 10 '25 22:03 r3yc0n1c

I would be interested in working on this. Could you please assign it to me?

parrth20 avatar Apr 11 '25 18:04 parrth20

I would be interested in working on this. Could you please assign it to me?

Nice! Can you collaborate with @r3yc0n1c on this?

theborakompanioni avatar Apr 11 '25 20:04 theborakompanioni

actually, I've resolved this already referring to #879 ... just clearing things up... will drop both the PRs soon. @parrth20 you can help us with some other issues maybe?

r3yc0n1c avatar Apr 11 '25 20:04 r3yc0n1c

actually, I've resolved this already referring to #879 ... just clearing things up... will drop both the PRs soon. @parrth20 you can help us with some other issues maybe?

Okay šŸ‘

parrth20 avatar Apr 11 '25 20:04 parrth20

Hi, I’d like to take this issue. Could you please assign it to me?

kunal-595 avatar Sep 27 '25 10:09 kunal-595