Proposal to Integrate RAI Protocol

stefan · July 12, 2021, 9:54pm

Summary

This proposal advocates for integrating the RAI protocol by adding the Reflexer Connector to the DSA Connector contracts. This would allow DeFi Smart Accounts to interact with the RAI protocol.

Proposal

RAI is a non-pegged stable asset and automated credit facility which recently implemented self-repaying loans. RAI can only be borrowed against ETH.

Reflexer built the necessary connector contracts so that RAI can be integrated in the Instadapp protocol. The Instadapp team also reviewed the code.

Reason

The Instadapp community would benefit from RAI self-repaying loans as well as the option to protect positions from liquidations, thus offering borrowers greater capital efficiency.

Code

github.com/Instadapp/dsa-connectors

Reflexer connector

Instadapp:main ← guifel:reflexer-connector

opened 11:05AM - 30 Mar 21 UTC

guifel

+696 -0

Hey, this is the connector PR. I tested every functions on a hardhat mainnet for…k. Breaking changes compared to the Maker connector: - Events `LogExitDai`, `LogDepositDai` and `LogWithdrawDai` were renamed - exitDai was renamed exit and its return event was adjusted accordingly. - Removed everything DSR related (Since we don't have a DSR) Note: - I'm not sure what is the `giveAddr` is and if it should be updated - I noticed a mismatch in the Maker contract between the event names [here](https://github.com/InstaDApp/dsa-connectors-new/blob/81a3f0b0b86407c912dc78d07b747d301499a424/contracts/connectors/makerdao/main.sol#L85) and [here](https://github.com/InstaDApp/dsa-connectors-new/blob/81a3f0b0b86407c912dc78d07b747d301499a424/contracts/connectors/makerdao/events.sol#L13). There is the same issue for the withdrawal event. In my code, I assumed it was a mistake. - Since in the other connectors you are harcoding the address of the mapping contract, I hardcoded the Reflexer mapping address as well in `getGebMappingAddress`. It will need to be updated once the mapping is deployed.

Result

This vote includes non-critical code changes and will be voted off-chain by the community.

The Reflexer Connector is added to the DSA Connectors master allowing it to be used with any DeFi Smart Account.

Vote On Snapshot

Status: Passed

Proposal on Snapshot

Seb_EthMonk · July 20, 2021, 8:45pm

Hey @stefan this looks good! Excited to see how we can develop and create use-cases for RAI. Currently we are voting in: IGP#2 - Streamlining and Growth Fund Grant

This will create a ‘streamlined’ integration process; after this vote an Instadapp Delegate will create a Snapshot proposal for RAI connector for a ‘streamlined’ integration.

stefan · July 26, 2021, 3:30pm

Saw IGP#2 passed, can’t wait to see the streamlined proposal!

Seb_EthMonk · July 29, 2021, 6:59pm

The Snapshot for this streamlined proposal is now live on Snapshot:
Proposal on Snapshot

Voting will be open for 3 days; a simply majority will enable the integration on the DeFi Smart Layer.

Seb_EthMonk · August 11, 2021, 5:45am

RAI Protocol integration passed and is active on the DeFi Smart Layer; you can read more about the RAI Connector commands in our developer docs here: Reflexer - Instadapp Developers

You can submit custom transactions using the RAI Connector on Instadapp Terminal