nucypher / ATxM

Automatic Transaction Machine 🏧 🔁
https://pypi.org/project/atxm/
GNU Affero General Public License v3.0
3 stars 3 forks source link

FaultTx/Faults are end states of tx execution #11

Closed derekpierre closed 8 months ago

derekpierre commented 8 months ago

Remove option for optionally clearing a tx that has faulted. The tx should be cleared since it is now considered completed.

After broadcasting, if the tx has faulted (determine by strategy) or reverted based on contract call failure, this is considered the end of the tx i.e. the tx machines work is done for that tx. Of course the on_fault hook should still be called. Since the hook is being called tracking faulty txs seems unnecessary.

Related to #20 .