-
Notifications
You must be signed in to change notification settings - Fork 83
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
Support "amend a pending transaction" #3045
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.
I'm not sure if the AmendTransaction
entity is necessary for this PR. The amendHash
appears to be the same as the original transaction's hash
on their feature, which makes me wonder if we can remove AmendTransaction
.
To implement the amends transaction feature in Neuron, we would need to:
- Add an "amend" button for each pending transaction.
- Create an "amend" page that allows users to increase the transaction fee.
- Fetch the minimal RBF fee and use it as the default amend fee.
- Send the amended transaction.
In my opinion, the AmendTransaction
is unnecessary for this feature and could be removed.
@homura Please have a review. |
/package |
I can not see amend feature in this package for mac-arm 64:https://github.com/nervosnetwork/neuron/actions/runs/8169785200?pr=3045 |
/package |
This feature is intended to solve the problem of pending transactions not being able to be packaged and therefore provides modification operations only in the pending state. As for the failure state not being visible, it may be related to the requirement. #Magickbase/neuron-public-issues#332 (comment) |
6.ckb asset account receive lost amend feature. 2024-03-11.17.09.27.mov |
All is same as before ,please check whether it is the testing package |
/package |
/package |
7.amend page verify custom price will cause automatic return to home page. 2024-03-12.15.14.40.mov |
/package |
4.verified |
/package |
8.verified |
Co-authored-by: Chen Yu <[email protected]>
issue: Magickbase/neuron-public-issues#332
Screen-2024-02-27-160430.mp4