feat: Supports amend transactions
issue: https://github.com/Magickbase/neuron-public-issues/issues/332
https://github.com/nervosnetwork/neuron/assets/11756869/649f35a7-1752-470b-ac47-1e571a3005da
@devchenyan Please review the above comments.
@homura @yanguoyu Please have a review.
@Keith-CY @homura Please have a review.
/package Packaging for test is done in 9072896666. @silySuper
Please merge latest code to this branch to sync normally @Keith-CY @devchenyan
Please merge latest code to this branch to sync normally @Keith-CY @devchenyan
Conflicted @devchenyan
Please merge latest code to this branch to sync normally @Keith-CY @devchenyan
Merged
neuron-ui: Module not found: Error: Can't resolve '@nervosnetwork/ckb-sdk-utils' in '/home/runner/work/neuron/neuron/packages/neuron-ui/src/components/AmendPendingTransactionDialog'
@nervosnetwork/ckb-sdk-utils was removed by PR https://github.com/nervosnetwork/neuron/pull/3149
/package Packageing failed in 9076288137. @silySuper
/package Packaging for test is done in 9076510260. @devchenyan
send transaction shows error.
Is this an occasional or mandatory error, I didn't encounter this error when I downloaded the package and tested it.
send transaction shows error.
main.log
Is this an occasional or mandatory error, I didn't encounter this error when I downloaded the package and tested it.
In my computer ,it is appear certainly,you can watch which informantion you need to check and I will offer you to solve it.
send transaction shows error.
It will be better to describe how to reproduce this issue. And it looks like the replace_fee is less than required. We can export the replacement transaction and the original transaction.
send transaction shows error.
main.log
It will be better to describe how to reproduce this issue. And it looks like the replace_fee is less than required. We can export the replacement transaction and the original transaction.
It was created during send transaction, I have send transaction successfully once ,then click refresh ,then send transaction again ,then show the error always,also deposit show this error too.It is not created by special operration,but by normal operation.I guess it may related to sync block.
sync has problem(sync progress see this issue:https://github.com/nervosnetwork/ckb/issues/4462),so I watch some situations which does not need transaction confirmed.
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
1.Why is the 'Price' button not available here, and also, how is the 'Suggested price' calculated? 2,3 OK
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
1.Why is the 'Price' button not available here, and also, how is the 'Suggested price' calculated? 2,3 OK
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
1.Why is the 'Price' button not available here, and also, how is the 'Suggested price' calculated? 2,3 OK
Because it is recommended that the price options routinely suggested are likely to be lower than the price needed to amend the transaction.And the user does not need to amend the suggested price by himself.
Get min_replace_fee via api and calculate it with the size of the transaction to get the 'Suggested price'
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
1.Why is the 'Price' button not available here, and also, how is the 'Suggested price' calculated? 2,3 OK
sync has problem(sync progress see this issue:nervosnetwork/ckb#4462 I watch some situations which does not need transaction confirmed.
![]()
1、amend transaction does not support selection of unit prices 2、“Unconfirmed Outputs are allowed in the transaction” is necessary 3、Too large a fee causes outputs to be larger than inputs, so cannot submit the transaction
1.Why is the 'Price' button not available here, and also, how is the 'Suggested price' calculated? 2,3 OK
Because it is recommended that the price options routinely suggested are likely to be lower than the price needed to amend the transaction.And the user does not need to amend the suggested price by himself.
Get min_replace_fee via api and calculate it with the size of the transaction to get the 'Suggested price'
OK.
In cell management:
1.when click submit the transaction,transaction hash is null ,but can copy,copy result is undefined.
In cell management: 1.when click submit the transaction,transaction hash is null ,but can copy,copy result is undefined.
![]()
fixed @silySuper
/package Packaging for test is done in 9121303973. @devchenyan
In Nervos Dao:
withdraw/create account amend can only be operated once,when amend twice it shows
unlock does not see amend button.
In Nervos Dao: withdraw/create account amend can only be operated once,when amend twice it shows
unlock does not see amend button.
![]()
- withdraw/create account amend can amend twice. Please make sure that the amend tx is new transaction, as the amended transaction will fail and be removed.
https://github.com/nervosnetwork/neuron/assets/11756869/232692b4-e6e3-46e5-8248-cb5711a01c3e
- The type of Unlcok is 'receive'. The treatment condition inputs-fee=outputs is not satisfied
/package Packaging for test is done in 9152602959. @silySuper
In Nervos Dao: withdraw/create account amend can only be operated once,when amend twice it shows
unlock does not see amend button.
- withdraw/create account amend can amend twice. Please make sure that the amend tx is new transaction, as the amended transaction will fail and be removed.
![]()
Screen-2024-05-17-113455.mp4 2. The type of Unlcok is 'receive'. The treatment condition inputs-fee=outputs is not satisfied
![]()
For the first problem,your video is different from mine.My operation is clicking amend when status is pending on amend transaction(not the first transaction). I guess is the status is failed in db,but shows pending in page cause this problem,so you can click amend but cannot fill in amend data.This can be considered to optimize when you are convenient later.




