-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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.
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
Add logging for Movement events #4147
base: main
Are you sure you want to change the base?
Conversation
@crStiv is attempting to deploy a commit to the unionbuild Team on Vercel. A member of the Team first needs to authorize it. |
thanks for the pr! please use |
@benluelo made changes as you asked and solved some ci errors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a few small comments. additionally, please refer to our CONTRIBUTING.md (we use conventional commits).
@benluelo made changes, btw you want me to change all the commit names or just the last one that I've changed already? |
yes, you will either need to squash or rename all of your commits (i recommend just squashing) |
Update main.rs Update main.rs Update main.rs
Added some useful logging for Movement IBC events so we can better track what's happening. Includes basic info like event type, block height, and tx hash, plus the key fields for each event type.
Fixes #3953.