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.

Sign up for GitHub

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

Jump to bottom

chore: adds testing and ensures event emit matches state #573

Closed

ckartik wants to merge 2 commits into main from ckartik/event-imps-pt1

Contributor

ckartik commented Jan 17, 2025

Resolves issue #572

ckartik added 2 commits

January 17, 2025 16:10


          chore: fixes edge case lack of logging

e22c77e


          chore: revert to main branch folder

ea932d1

ckartik requested review from shaspitz and Mikelle

January 17, 2025 21:15

Contributor Author

ckartik commented Jan 23, 2025

Closing, since the state and emitted events already reverts when the transaction fails.

ckartik closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet