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

Remove expiration for cached rates #32

Merged

JanKuczma merged 6 commits into main from refactor-rates

Aug 13, 2024

Collaborator

JanKuczma commented Aug 13, 2024 •

edited

Loading

Remove cache expiration timestamp logic.

All rates must be from the current block.
Add getter method for token rates providers contract addresses


          remove expiration for cached rates

2603ed3

JanKuczma requested a review from woocash2

August 13, 2024 09:33

JanKuczma added 4 commits

August 13, 2024 11:53


          rename method

be58f48


          add getter for rate providers

3108ed9


          auto deref

d28c083


          clippy

01095e0

woocash2 reviewed

View reviewed changes

amm/traits/stable_pool.rs Outdated Show resolved Hide resolved

woocash2 reviewed

View reviewed changes

amm/traits/stable_pool.rs Outdated Show resolved Hide resolved

woocash2 reviewed

View reviewed changes

amm/contracts/stable_pool/lib.rs Outdated Show resolved Hide resolved

woocash2 reviewed

View reviewed changes

amm/contracts/stable_pool/lib.rs Show resolved Hide resolved

woocash2 approved these changes

View reviewed changes

Contributor

woocash2 left a comment

LGTM, few low importance nits found


          compute block_no inside TokenRate

21e3cbf

JanKuczma merged commit 2925860 into main

1 check passed

JanKuczma deleted the refactor-rates branch

August 13, 2024 12:13

woocash2 reviewed

View reviewed changes

amm/contracts/stable_pool/token_rate.rs Show resolved Hide resolved

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

Labels

None yet