Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Governance Refactor and Bug Fixes #11

Merged
merged 2 commits into from Mar 22, 2022
Merged

Governance Refactor and Bug Fixes #11

merged 2 commits into from Mar 22, 2022

Conversation

binpmxstn
Copy link
Contributor

  • Added accrueInterest function to be called during unlock cycle
  • Update protocol initialization function to be called more than once
  • Refactored addContract function to restrict replacing an existing contract
  • Fixed bug in getReporter function
  • Fixed bug in getResolutionInfoFor function
  • Added zero value checks on Processor, cxToken, cxTokenFactory, Reporter, Finalization, Resolvable, Unstakable, CoverBase, CoverProvision, CoverReassurance, CoverStake, LiquidityEngine, VaultBase, WithFlashLoan, Policy, PolicyAdmin, StoreBase, BaseLibV1, StakingPoolLibV1, and more.
  • Created events.md file
  • Dropped redundant function Finalized from ICxToken
  • Dropped redundant function PriceUpdated from IPriceDiscovery
  • Refactored namespaces to keen into account both active and historical cover statuses. Functions refactored and added getCoverStatusOf, getStatusOf, getCoverStatusKey, getCoverStatusOfKey,
  • Fixed slither and NPM analyzer issues
  • Refactored unit tests
  • Refactored deployment script, composer, and utils
  • Updated ABIs
  • Updated documentation
  • Updated keys

- Added `accrueInterest` function to be called during unlock cycle
- Update protocol initialization function to be called more than once
- Refactored `addContract` function to restrict replacing an existing contract
- Fixed bug in `getReporter` function
- Fixed bug in `getResolutionInfoFor` function
- Added zero value checks on `Processor`, `cxToken`, `cxTokenFactory`, `Reporter`, `Finalization`, `Resolvable`, `Unstakable`, `CoverBase`, `CoverProvision`, `CoverReassurance`, `CoverStake`, `LiquidityEngine`, `VaultBase`, `WithFlashLoan`, `Policy`, `PolicyAdmin`, `StoreBase`, `BaseLibV1`, `StakingPoolLibV1`, and more.
- Created `events.md` file
- Dropped redundant function `Finalized` from `ICxToken`
- Dropped redundant function `PriceUpdated` from `IPriceDiscovery`
- Refactored namespaces to keen into account both active and historical cover statuses. Functions refactored and added `getCoverStatusOf`, `getStatusOf`, `getCoverStatusKey`, `getCoverStatusOfKey`,
- Fixed slither and NPM analyzer issues
- Refactored unit tests
- Refactored deployment script, composer, and utils
- Updated ABIs
- Updated documentation
- Updated keys
@binpmxstn binpmxstn merged commit 0bcb608 into develop Mar 22, 2022
@binpmxstn binpmxstn deleted the fix/governance branch March 23, 2022 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant