Skip to content

refactor: change to soroban client & fix event fetching #174

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
Merged
Teolhyn merged 3 commits into from
Sep 15, 2025
Merged

refactor: change to soroban client & fix event fetching #174

Teolhyn merged 3 commits into from
Sep 15, 2025

Conversation

@Teolhyn
Copy link
Member

@Teolhyn Teolhyn commented Sep 15, 2025 •
edited
Loading

Summary of Changes

Changed event fetching in liquidation bot to use Soroban client. Also, added built in event filters so that it only fetches events of interest.

Considerations

This does not mean the liquidation bot works as expected. Loan parsing from events has to be rebuilt for it to work.

Related Issue

Please include a link to the relevant issue for context.

Author checklist

  1. Does your pull request change contract endpoint?
  • Yes, changes in loan_manager
    • Test was added in loan_manager.rs
  • Yes, changes in loan_pool
    • Test was added in loan_pool.rs
  • No
  1. Does your pull request demand a new deployment of contracts
  • Yes
  • No

kovipu
kovipu approved these changes Sep 15, 2025
View reviewed changes
Copy link
Member

@kovipu kovipu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Just rustfmt

.env.local.backup Outdated
PUBLIC_CONTRACT_ADDRESS_USDC=CAXBQNDXJWLDT6J6L2IPRDRUSHC6Z7WGH35RPZCMUMU2A3NYENRUXRQE
PUBLIC_CONTRACT_ADDRESS_EURC=CDZ7SGEC4DWLPYQUUAN6MHTBCHAS3NVLDJMK6OGZOOIWWDIWTQFTSFAX
PUBLIC_ISSUER_ADDRESS_USDC=GAA5BNGYQH632CHGCKMDTYN35CXZSWZR3NKNE2WTGUF3A5QNPPZ2Y322
PUBLIC_ISSUER_ADDRESS_EURC=GAA5BNGYQH632CHGCKMDTYN35CXZSWZR3NKNE2WTGUF3A5QNPPZ2Y322
Copy link
Member

@kovipu kovipu Sep 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file needed?

@Teolhyn Teolhyn changed the title draft: refactor: change to soroban client & fix event fetching refactor: change to soroban client & fix event fetching Sep 15, 2025
@Teolhyn Teolhyn merged commit 427cf29 into main Sep 15, 2025
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kovipu kovipu kovipu approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Teolhyn @kovipu