Follow-up from "Feemanager Watchdog"
The following discussions from !4536 (merged) should be addressed:
-
@ChrisSchinnerl started a discussion: (+3 comments) I think instead of having this partialTxns field it might be easier to introduce another entry type for the final entry of a transaction.
Then when you loop over the entries and you encounter a transaction, you continue until you find a final piece of a transaction and call
managedBuildTransaction(ets []entryTransaction) (types.Transaction, error)
This should work since all the transaction entries are right next to each other in the persist anyway right?
-
@ChrisSchinnerl started a discussion: (+2 comments) This is tricky. Cause when we send the siacoins and then crash before persisting the transaction, we will try to send the fee again right? So we should build the transaction without sending it, then monitor/persist it and finally submit it to the tpool.