Closed saeta-eth closed 1 month ago
Latest commit: 37418726fda5728c10a6d44e4f8024827a37b410
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
This PR addresses an issue with the useSafeTransactionStatus hook where transaction status was not being accurately determined. The hook has been updated to use the Safe 1.4.1 ABI to check transaction logs, ensuring accurate identification of transaction success or failure.
This transaction worked as it uses the old Safe ABI.
This transaction did not work as it uses the Safe 1.4.1 ABI. This PR resolves that issue.
Closes https://linear.app/usecannon/issue/CAN-555/safe-status-check-is-not-working-on-newer-txs