Skip to content
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

Resolve etherscan warning on all etherscan ctoken interactions #23

Open
sriyantra opened this issue May 18, 2022 · 0 comments
Open

Resolve etherscan warning on all etherscan ctoken interactions #23

sriyantra opened this issue May 18, 2022 · 0 comments
Labels
enhancement New feature or request nice to have

Comments

@sriyantra
Copy link
Collaborator

Origin of warning: revert on checkpointInterest() call attempt for use with reactive interest rate models

Solution: a storage bool for when checkpointInterest() call is attempted, set to true upon success so that future ctoken calls don't have an etherscan execution error. This could be implemented for future deployments so it doesn't have to potentially compromise existing ctokens if there were any issues. The standard to add a new storage var in proxies is to append a new struct version #

This would most likely be used for future ctoken deployments and not upgraded on current ctokens in order to maintain safety and minimal changes

@sriyantra sriyantra added enhancement New feature or request nice to have labels May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request nice to have
Projects
None yet
Development

No branches or pull requests

1 participant