feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] #225

cgewecke · 2022-03-18T15:31:53Z

Create deprecated folder for contracts we longer need or use.
Deprecate NavIssuanceModule (in favor of CustomOracleNAVIssuanceModule)
Deprecate PerpV2LeverageModule (in favor of PerpV2LeverageModuleV2)
Deprecate AaveGovernanceAdapter (in favor of AaveGovernanceV2Adapter)
Deprecate UniswapYieldHook
Deprecate all AaveV1 contracts, external ABIs and associated fixtures
Deprecate all single use token migration contracts

bweick · 2022-03-21T20:14:53Z

I don't think we need the IssuanceModule update since that was already done in a #201 by Pranav.

bweick · 2022-03-21T20:15:47Z

In favor of deprecating all AaveV1 and token migration contracts. Maybe we can link to an example of a token migration contract in the README?

This reverts commit e4fe2a2.

bweick · 2022-03-24T18:04:13Z

test/product/uniswapYieldHook.spec.ts

 import { Account } from "@utils/test/types";
 import { ZERO, ADDRESS_ZERO } from "@utils/constants";
-import { NAVIssuanceCaller, NavIssuanceModule, SetToken, UniswapYieldHook } from "@utils/contracts";
+import {


We should move the UniswapYieldHook to the deprecated folder

cgewecke · 2022-03-24T18:55:29Z

🎉 This PR is included in version 0.3.0-hhat.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

# [0.3.0](v0.2.0...v0.3.0) (2022-03-24) ### Features * **deprecation:** Consolidate or remove deprecated contracts [SIM-124] ([#225](#225)) ([579852a](579852a))

# 1.0.0 (2022-10-04) ### Bug Fixes * **adapter:** Support Curve stableswap pools ([SetProtocol#253](https://github.com/IndexCoop/index-protocol/issues/253)) ([777ed94](777ed94)) * add proper tests, require statements for sellTokenForToken ([5c84995](5c84995)) * address review feedback ([5c6fcfc](5c6fcfc)) * **build:** Upgrade eslint-plugin-jsdoc to latest ([SetProtocol#228](https://github.com/IndexCoop/index-protocol/issues/228)) ([f5758d2](f5758d2)) * **interfaces:** add IAirdropModule, IClaimModule, and IWrapModuleV2 ([SetProtocol#252](https://github.com/IndexCoop/index-protocol/issues/252)) ([6999e84](6999e84)) * **interfaces:** Add protocol interfaces for set-v2-strategies [SIM-124] ([SetProtocol#226](https://github.com/IndexCoop/index-protocol/issues/226)) ([0bd5ca4](0bd5ca4)) * **IPerpV2BasisTradingModule:** Improve javadocs [SetProtocol#246](https://github.com/IndexCoop/index-protocol/issues/246) ([3129b37](3129b37)) * **NotionalTradeModule:** audit adjustments ([de39ee1](de39ee1)) * **PerpV2BasisTradingModule:** bytecode size limit bug ([SetProtocol#200](https://github.com/IndexCoop/index-protocol/issues/200)) ([ddfab38](ddfab38)) * **PerpV2BasisTradingModule:** Expose getter to fetch updated settled funding ([SetProtocol#244](https://github.com/IndexCoop/index-protocol/issues/244)) ([f215147](f215147)) * **PerpV2BasisTradingModule:** External audit fixes for low-risk findings and Internal audit fixes ([SetProtocol#232](https://github.com/IndexCoop/index-protocol/issues/232)) ([d7d37da](d7d37da)), closes [SetProtocol#243](https://github.com/IndexCoop/index-protocol/issues/243) * **PerpV2LeverageModuleV2:** Round up during withdraw ([SetProtocol#249](https://github.com/IndexCoop/index-protocol/issues/249)) ([7780963](7780963)) * remove incorrect require check for sellEthForTokenToUniswapV3 ([7f8048d](7f8048d)) * sellEthForTokenToUniswapV3 impl and address review comments ([73ea7b3](73ea7b3)) * **SlippageIssuanceModule:** Fix issuance & redemption flows to not update position units ([SetProtocol#254](https://github.com/IndexCoop/index-protocol/issues/254)) ([9be3d1b](9be3d1b)) * **tests:** Add Hardhat deployment subgraph test scripts ([SetProtocol#242](https://github.com/IndexCoop/index-protocol/issues/242)) ([0aa0d14](0aa0d14)) * **utils:** Port StringArrayUtils from set-v2-strategies [SIM-124] ([SetProtocol#227](https://github.com/IndexCoop/index-protocol/issues/227)) ([dbf6c04](dbf6c04)) ### Features * **adapter:** Add Curve trade adapter [SIM-173] ([SetProtocol#238](https://github.com/IndexCoop/index-protocol/issues/238)) ([f697783](f697783)) * **adapter:** Add RGT to Tribe migration adapter [SIM-52] ([SetProtocol#188](https://github.com/IndexCoop/index-protocol/issues/188)) ([ca5c2ab](ca5c2ab)) * add direct RFQT fill suport to ZeroExApiAdapter [SIM-230] ([SetProtocol#241](https://github.com/IndexCoop/index-protocol/issues/241)) ([f4b976e](f4b976e)) * **BytesArrayUtils:** Add BytesArrayUtils library ([SetProtocol#248](https://github.com/IndexCoop/index-protocol/issues/248)) ([7b35392](7b35392)) * **deprecation:** Consolidate or remove deprecated contracts [SIM-124] ([SetProtocol#225](https://github.com/IndexCoop/index-protocol/issues/225)) ([579852a](579852a)) * implement sellTokenForEthToUniswapV3 and sellEthForTokenToUniswapV3 ([2296f6e](2296f6e)) * initial stab at Uniswap V3 adaptor ([c25665b](c25665b)) * **Intefaces:** Add basis trading module interfaces ([SetProtocol#237](https://github.com/IndexCoop/index-protocol/issues/237)) ([c7af322](c7af322)) * **module:** Notional Trade Module ([SetProtocol#251](https://github.com/IndexCoop/index-protocol/issues/251)) ([49bdf46](49bdf46)) * **publishing:** Auto publish repository to npm in CI [SIM-123] ([SetProtocol#203](https://github.com/IndexCoop/index-protocol/issues/203)) ([8ddbbf1](8ddbbf1)) * **reorg:** Re-org modules into v1 and v2 directories ([SetProtocol#230](https://github.com/IndexCoop/index-protocol/issues/230)) ([d43df0c](d43df0c)) * **UniswapV3ExchangeAdapterV2:** Add UniswapV3ExchangeAdapterV2 exchange adapter ([SetProtocol#240](https://github.com/IndexCoop/index-protocol/issues/240)) ([b4d95ca](b4d95ca)) * verify path for multiple hops ([06ca9b6](06ca9b6)) * verify uniswap path for single hops WIP ([7905d73](7905d73))

# 1.0.0 (2022-10-05) ### Bug Fixes * **adapter:** Support Curve stableswap pools ([SetProtocol#253](https://github.com/IndexCoop/index-protocol/issues/253)) ([777ed94](777ed94)) * add proper tests, require statements for sellTokenForToken ([5c84995](5c84995)) * address review feedback ([5c6fcfc](5c6fcfc)) * **build:** Upgrade eslint-plugin-jsdoc to latest ([SetProtocol#228](https://github.com/IndexCoop/index-protocol/issues/228)) ([f5758d2](f5758d2)) * **interfaces:** add IAirdropModule, IClaimModule, and IWrapModuleV2 ([SetProtocol#252](https://github.com/IndexCoop/index-protocol/issues/252)) ([6999e84](6999e84)) * **interfaces:** Add protocol interfaces for set-v2-strategies [SIM-124] ([SetProtocol#226](https://github.com/IndexCoop/index-protocol/issues/226)) ([0bd5ca4](0bd5ca4)) * **IPerpV2BasisTradingModule:** Improve javadocs [SetProtocol#246](https://github.com/IndexCoop/index-protocol/issues/246) ([3129b37](3129b37)) * **NotionalTradeModule:** audit adjustments ([de39ee1](de39ee1)) * **PerpV2BasisTradingModule:** bytecode size limit bug ([SetProtocol#200](https://github.com/IndexCoop/index-protocol/issues/200)) ([ddfab38](ddfab38)) * **PerpV2BasisTradingModule:** Expose getter to fetch updated settled funding ([SetProtocol#244](https://github.com/IndexCoop/index-protocol/issues/244)) ([f215147](f215147)) * **PerpV2BasisTradingModule:** External audit fixes for low-risk findings and Internal audit fixes ([SetProtocol#232](https://github.com/IndexCoop/index-protocol/issues/232)) ([d7d37da](d7d37da)), closes [SetProtocol#243](https://github.com/IndexCoop/index-protocol/issues/243) * **PerpV2LeverageModuleV2:** Round up during withdraw ([SetProtocol#249](https://github.com/IndexCoop/index-protocol/issues/249)) ([7780963](7780963)) * remove incorrect require check for sellEthForTokenToUniswapV3 ([7f8048d](7f8048d)) * sellEthForTokenToUniswapV3 impl and address review comments ([73ea7b3](73ea7b3)) * **SlippageIssuanceModule:** Fix issuance & redemption flows to not update position units ([SetProtocol#254](https://github.com/IndexCoop/index-protocol/issues/254)) ([9be3d1b](9be3d1b)) * **tests:** Add Hardhat deployment subgraph test scripts ([SetProtocol#242](https://github.com/IndexCoop/index-protocol/issues/242)) ([0aa0d14](0aa0d14)) * **utils:** Port StringArrayUtils from set-v2-strategies [SIM-124] ([SetProtocol#227](https://github.com/IndexCoop/index-protocol/issues/227)) ([dbf6c04](dbf6c04)) ### Features * **adapter:** Add Curve trade adapter [SIM-173] ([SetProtocol#238](https://github.com/IndexCoop/index-protocol/issues/238)) ([f697783](f697783)) * **adapter:** Add RGT to Tribe migration adapter [SIM-52] ([SetProtocol#188](https://github.com/IndexCoop/index-protocol/issues/188)) ([ca5c2ab](ca5c2ab)) * add direct RFQT fill suport to ZeroExApiAdapter [SIM-230] ([SetProtocol#241](https://github.com/IndexCoop/index-protocol/issues/241)) ([f4b976e](f4b976e)) * **BytesArrayUtils:** Add BytesArrayUtils library ([SetProtocol#248](https://github.com/IndexCoop/index-protocol/issues/248)) ([7b35392](7b35392)) * **deprecation:** Consolidate or remove deprecated contracts [SIM-124] ([SetProtocol#225](https://github.com/IndexCoop/index-protocol/issues/225)) ([579852a](579852a)) * implement sellTokenForEthToUniswapV3 and sellEthForTokenToUniswapV3 ([2296f6e](2296f6e)) * initial stab at Uniswap V3 adaptor ([c25665b](c25665b)) * **Intefaces:** Add basis trading module interfaces ([SetProtocol#237](https://github.com/IndexCoop/index-protocol/issues/237)) ([c7af322](c7af322)) * **module:** Notional Trade Module ([SetProtocol#251](https://github.com/IndexCoop/index-protocol/issues/251)) ([49bdf46](49bdf46)) * **publishing:** Auto publish repository to npm in CI [SIM-123] ([SetProtocol#203](https://github.com/IndexCoop/index-protocol/issues/203)) ([8ddbbf1](8ddbbf1)) * **reorg:** Re-org modules into v1 and v2 directories ([SetProtocol#230](https://github.com/IndexCoop/index-protocol/issues/230)) ([d43df0c](d43df0c)) * **UniswapV3ExchangeAdapterV2:** Add UniswapV3ExchangeAdapterV2 exchange adapter ([SetProtocol#240](https://github.com/IndexCoop/index-protocol/issues/240)) ([b4d95ca](b4d95ca)) * verify path for multiple hops ([06ca9b6](06ca9b6)) * verify uniswap path for single hops WIP ([7905d73](7905d73))

cgewecke added 4 commits March 18, 2022 07:56

Remove NAVIssuanceModule (in favor of CustomOracleNAVIssuanceModule)

cef2cca

Rename DebtIssuanceModuleV2 to IssuanceModule

e4fe2a2

Remove AaveGovernanceAdapter (in favor of AaveGovernanceV2Adapter)

c19cb8d

Fix aaveGovernanceV2Adapter test

c904c3c

cgewecke changed the title ~~feat(deprecation): Consolidate or remove deprecated contracts [SIM-123]~~ feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] Mar 18, 2022

This was referenced Mar 19, 2022

fix(interfaces): Add protocol interfaces for set-v2-strategies [SIM-124] #226

Merged

Import interfaces and lib utils from set-protocol-v2 [SIM-124] SetProtocol/set-v2-strategies#22

Merged

cgewecke added 3 commits March 22, 2022 06:40

Revert "Rename DebtIssuanceModuleV2 to IssuanceModule"

6e5344d

This reverts commit e4fe2a2.

Deprecate AaveV1 and token migrations

8bbaa1f

Remove PerpV2LeverageModule (in favor of PerpV2LeverageModuleV2)

d92c68a

cgewecke force-pushed the chris/remove-unused-contracts branch from c6ee84d to d92c68a Compare March 22, 2022 16:47

bweick reviewed Mar 24, 2022

View reviewed changes

bweick approved these changes Mar 24, 2022

View reviewed changes

Deprecate UniswapYieldHook

c7382fa

cgewecke merged commit 579852a into master Mar 24, 2022

cgewecke deleted the chris/remove-unused-contracts branch March 24, 2022 18:48

cgewecke added the released on @hardhat label Mar 24, 2022

cgewecke pushed a commit that referenced this pull request Mar 24, 2022

chore(release): 0.3.0 [skip ci]

c2dbb92

# [0.3.0](v0.2.0...v0.3.0) (2022-03-24) ### Features * **deprecation:** Consolidate or remove deprecated contracts [SIM-124] ([#225](#225)) ([579852a](579852a))

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] #225

feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] #225

cgewecke commented Mar 18, 2022 •

edited

Loading

bweick commented Mar 21, 2022

bweick commented Mar 21, 2022

bweick Mar 24, 2022

cgewecke commented Mar 24, 2022

feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] #225

feat(deprecation): Consolidate or remove deprecated contracts [SIM-124] #225

Conversation

cgewecke commented Mar 18, 2022 • edited Loading

bweick commented Mar 21, 2022

bweick commented Mar 21, 2022

bweick Mar 24, 2022

Choose a reason for hiding this comment

cgewecke commented Mar 24, 2022

cgewecke commented Mar 18, 2022 •

edited

Loading