Comptroller Upgrade - Githubissues

InverseFinance / anchor

BSD 3-Clause "New" or "Revised" License

28 stars 25 forks source link

Comptroller Upgrade #12

Closed nourharidy closed 3 years ago

nourharidy commented 3 years ago

remove updateCompBorrowIndex, disabling future borrowing rewards for all markets
INV rewards to be transferred from treasury address up to an allowance, INV will no longer be stored in the comptroller

This comptroller will replace the existing mainnet comptroller via a proxy upgrade. Proxy address: 0x4dCf7407AE5C07f8681e1659f626E114A7667339 Current comptroller implementation address (to be replaced): 0x2c0edf1f7dbcdb347ed8ce626d4de2221f1d76a2

nourharidy commented 3 years ago

Could you explain the effect of removing updateCompBorrowIndex in more detail? For instance, are there any side-effects of removing updateCompBorrowIndex but keeping distributeBorrowComp? I need to review the whole codebase in more detail but using this PR as an opportunity to get familiar with it.

I kept distributeBorrowComp in order to allow the distribution of existing borrowing rewards. Only future rewards are disabled.