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

Add an owner to StakeManager #427

Closed
wants to merge 2 commits into from

Conversation

goran-ethernal
Copy link
Contributor

This PR adds an owner to the StakeManager contract.

Stefan-Ethernal and others added 2 commits November 9, 2023 12:29
* feat(child): add `GenesisProxy` and dummy deployment scripts

* feat: prepare for 1.1

* fix: slither

* feat: add initial `DeployChildContracts.s.sol`

* fix: clean up 2/3

* docs: update docs

* fix: restore `EPOCH_SIZE` in VS

* fix: rm `NetworkParams` and `ForkParams`

* fix: clean up 3/3

* fix: sliter

* Premine function implementation

* Fix slither

* Unit tests

* Use correct forge-std revision

* Address comments (part 1)

* Remove blank line

* Pass tests again

* Introduce genesisBalances field for storing premine balances

* Rename premine to addGenesisBalance

* Remove nativeTokenRoot function and make field public

* Track genesis balances in the CustomSupernetManager

* Rename event

* Remove getGenesisBalance function

* Minor fix

---------

Co-authored-by: Zero Ekkusu <[email protected]>
* contracts fix for l1

* stakeManager and stakeManagerStakingData as one contract

* fix stakeManager imports

* Remove unnecessary contracts

* Remove patch

* Add TODOs to ValidatorSet contract

* Remove DeployCustomSupernetManager from rootchain deploy script

* Update imports for StakeManager

* Remove unused constants

* Fixing StakeManager tests

* change StartingValidator to GenesisValidator

* ValidatorSet merged into StakeManager and EpochManager

* Remove NativeERC20 and rename Mintable

* deleted unnecessary initialize attributes

* Fix some StakeManager tests, remove EIP1559Burn, and some deployment scripts

* Fix ChildERC20Predicate hardhat tests

* Slither fixes regarding epochSize in EpochManager

* Split requires

* Update EpochManager tests (part 1)

* More test fixes

* Fix compilation errors

* Fix some EpochManager tests

* EpochManager fix

* Fix slither

* EpochManager fix

* Stake fix

* Fix EpochManager tests

* Fix StakeManager slither

* Suppress Merkle slither warning

* Suppres slither warnings

* Disable slither check for _getCurrentSnapshotId

* Fix reentrancy in StakeManager

* Remove unused field in StakeManager

* Remove generateRegistrationSignature

* Cleanup

* Folder reorganization

* Store validator blsKey

---------

Co-authored-by: Dusan <[email protected]>
Co-authored-by: Goran Rojovic <[email protected]>
Copy link

sonarcloud bot commented Nov 28, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

2 participants