Add custom Fee, fix a problem with isOnline in getsettings, note Automation only for chat related tx #146
Merged
duke
merged 4 commits from lucretius/SilentDragonLite:dev
into dev
3 months ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'lucretius/SilentDragonLite:dev'
Deleting a branch is permanent. It CANNOT be undone. Continue?
This PR adds the option to use a custom fee for transactions.
Checkbox on the Sentab has been added.
The field "mining fee" is only writable if the box is checked, otherwise it is read only and the default fee is used.
Added a note on the tx confirmation window if a custom fee is used.
It also fixes a problem with the bool isOnline where there was one too many data formats.
Also the note Automation only triggers now for chat related tx.
Add custom Fee, fix a problem with isOnline in getsettingsto Add custom Fee, fix a problem with isOnline in getsettings, note Automation only for chat related tx 3 months agoe299fe33e9
into dev 3 months ago@lucretius looks good, thank you. I think we may want to add a "sanity check" for custom fees, for the case when a user probably makes a typo for the fee. For example, if the user wants to use a fee which is larger than the amount being sent (in the case where the amount being sent is non-zero and above the default fee), there should be a popup that says "Are you really sure about that?" or something like that.
e299fe33e9
.