Open tequdev opened 4 days ago
Its improperly named I agree. The problem is, we cannot switch the field or every current client will break.
base_fee
is the base + hooks + memos + hook parameters
base_fee_no_hooks
is just the base
We can rename it if you'd like
Issue Description
The
base_fee_no_hooks
field included in the return value of thefee
command does not include fees other than Hooks fees, such as dynamic fees for Memo fields.https://github.com/Xahau/xahaud/blob/79d83bd424db07a51fa3d1ea41a01ecfdf7af99c/src/ripple/app/misc/impl/TxQ.cpp#L2079
This is an issue that is not urgent, but I will keep a note of it.
Steps to Reproduce
result
Expected Result
It includes all fees other than Hooks execution fees.
Actual Result
Environment
Version: 2024.9.11-release+985
Supporting Files