search

trustline-inc / probity

A transparent asset-based lending protocol implemented for the EVM.
https://docs.trustline.co
Other
3 stars 1 forks source link

AssetManagers still have onlyBy("whitelisted") modifier #391

Closed shine2lay closed 2 years ago

shine2lay commented 2 years ago

Since we talked about making vault engine base version without whitelisted modifier, i think we should remove them from VP asset manager as well, otherwise users will not be able to deposit funds without the whitelisting feature

mrosendin commented 2 years ago

should we have versions that inherit the base asset manager with whitelisting?

shine2lay commented 2 years ago

I personally don't see a need for it, if you do, let me know