Conversation
|
The contracts CI is failing on the rustfmt format check. This isn't a logic issue, just formatting. Run this in the contracts directory and push: cd contracts
cargo fmtThen commit and push the formatting changes. The CI should pass after that. |
|
Backend CI is failing with a Jest environment error: This usually means a test file has async operations that outlive the test suite — either missing To fix: check if any test files in your PR have async operations in top-level scope without proper Contracts CI is also failing on |
ogazboiz
left a comment
There was a problem hiding this comment.
Two failures to fix:
Contracts CI: Failing under 30s, which is a cargo fmt issue.
cd contracts
cargo fmtBackend CI: eventIndexer.test.ts failure. Rebase on main to pick up the recent test fix:
git fetch upstream
git rebase upstream/mainCommit, format, and push when done.
|
The codebase issues on main have been resolved and all CI checks are passing now. Please rebase your branch to pull in the latest changes before continuing. Thanks for your patience. |
ogazboiz
left a comment
There was a problem hiding this comment.
Has merge conflicts. Imports use bare module paths without .js extensions (inconsistent with ESM codebase). Every handler uses event: any with no validation. No migration files included for the ~9 new tables referenced. The Rust code appends a second impl block which won't compile. Needs significant rework.
|
heads up, a few important changes just landed on main that affect your PR:
please rebase on latest main: git fetch upstream
git rebase upstream/main
git push --force-with-lease |
📝 Description
Overview
This PR extends the event indexer to subscribe to LendingPool, RemittanceNFT, and MultisigGovernance contract events.
Key Features
Closes #437