Skip to content
This repository was archived by the owner on Apr 2, 2026. It is now read-only.

feat: fixes, cleanup and tests#29

Merged
lekhovitsky merged 50 commits intonextfrom
getter-updates
Mar 2, 2025
Merged

feat: fixes, cleanup and tests#29
lekhovitsky merged 50 commits intonextfrom
getter-updates

Conversation

@lekhovitsky
Copy link
Copy Markdown
Collaborator

No description provided.

@lekhovitsky lekhovitsky force-pushed the next branch 2 times, most recently from c0038fc to 6c1b6ac Compare January 31, 2025 11:45
lekhovitsky and others added 12 commits February 10, 2025 19:11
In this commit:
- upgrade deps
- CF and CC no longer expect ACL as constructor argument
- credit factory reads WETH from AP in runtime and not in constructor
- aliased loss policy conforms to default constructor signature
- add missing serialization
- revert on non-v3 minor version
- `getLatestMinorVersion` and `getLatestPatchVersion` accept non-exact versions
feat: add `DefaultDegenNFT`, fix some bugs and add configuration tests
feat: ccg proposal naming change + recovery mode
In this PR:
- updatable price feeds now live in `PriceFeedStore` instead of `PriceOracleV3`
- `PriceFeedStore` allows to configure price feeds it owns (think `setLimiter` in LP price feeds)
- some fixes and improvements in `PriceOracleFactory`
  - allow reserve price feed to be `ZeroPriceFeed` even if it's not added to allowed price feeds for a token
  - fix migration from legacy price oracle
  - gas-optimize migration of price feeds
- minor fixes in `NestedPriceFeeds` library
- general refactoring of `PriceFeedStore` (better execution flow, NatSpec, etc.)
- minor changes related to `ILossPolicy` interface upgrade
* feat: add `removePriceFeed` to the store

Additionally, removes the obsolete `author` field from the `PriceFeedInfo` struct.

* fix: cleanup `MarketConfiguratorFactory` + unit tests

* feat: cleanup `InstanceManager` + unit tests

* fix: generalized legacy periphery contracts migration

* feat: cleanup `AddressProvider` + unit tests

* chore: update bindings
- `MarketConfigurator`'s `authorizeFactory` and `unauthorizeFactory` no longer revert on no-ops
- `CreditFactory`'s `upgradeCreditConfigurator` quotes all tokens in legacy credit managers
- `CreditFactory`'s `allowAdapter` now automatically unauthorizes previous adapter for the target contract if it was set
- `MarketConfiguratorLegacy` no longer authorizes factories for contracts that are not used after the migration
- `LegacyParams` now includes `priceOracle` since we've moved away from the idea of deploying a price oracle for each pool before migration
Also remove it in contracts where it wasn't that needed
- make system more robust against instance operator strategically placing `uploadBytecode` and `submitAuditReport` within governance transactions
- cleaner separation between system and public domains (they are either system or public and it's one-way now, only public contracts have owner)
- more comprehensible protection against domain squatting by compromised auditor
- better contract type and version validation and control
- Recovery mode message is now chain-specific
- In recovery mode, self-calls are not skipped
- `disableRecoveryMode` must be the only call in the batch
@lekhovitsky lekhovitsky changed the title feat: getter updates feat: fixes, cleanup and tests Mar 2, 2025
@lekhovitsky lekhovitsky merged commit 82b8613 into next Mar 2, 2025
1 check passed
@lekhovitsky lekhovitsky deleted the getter-updates branch March 2, 2025 11:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants