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

bridge commands relocation #19

Merged
merged 11 commits into from
Nov 28, 2023
Merged

bridge commands relocation #19

merged 11 commits into from
Nov 28, 2023

Conversation

dusannosovic-ethernal
Copy link

Description

This PR Includes commands relocation

Relocation inludes rootchain and sidechain commands.

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • Hotfix (change that solves an urgent issue, and requires immediate attention)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)

Breaking changes

Please complete this section if any breaking changes have been made, otherwise delete it

Checklist

  • I have assigned this PR to myself
  • I have added at least 1 reviewer
  • I have added the relevant labels
  • I have updated the official documentation
  • I have added sufficient documentation in code

Testing

  • I have tested this code with the official test suite
  • I have tested this code manually

@dusannosovic-ethernal dusannosovic-ethernal changed the base branch from l1-chain to staking November 24, 2023 07:50
@Stefan-Ethernal Stefan-Ethernal changed the base branch from staking to l1-chain November 28, 2023 10:28
@goran-ethernal goran-ethernal deleted the commands-relocation branch November 28, 2023 10:35
@Stefan-Ethernal Stefan-Ethernal restored the commands-relocation branch November 28, 2023 10:36
@goran-ethernal goran-ethernal marked this pull request as ready for review November 28, 2023 10:57
@goran-ethernal goran-ethernal merged commit 6b95bb1 into l1-chain Nov 28, 2023
9 of 13 checks passed
@goran-ethernal goran-ethernal deleted the commands-relocation branch November 28, 2023 11:07
goran-ethernal added a commit that referenced this pull request Dec 6, 2023
* apply patch

* Fix for unit tests

* lint fix

* lint fix

* contracts fix

* makefile fix

* build fix

* fix some imports

* Lint fix

* comment fixs

* fix imports

* Update command/genesis/genesis.go

Co-authored-by: Stefan Negovanović <93934272+Stefan-Ethernal@users.noreply.github.com>

* Update command/genesis/params.go

Co-authored-by: Stefan Negovanović <93934272+Stefan-Ethernal@users.noreply.github.com>

* fix comments

* Linter fixes

* deleted supernet package

* stake fix

* stake fix 2

* comments fix

* Use new version of contracts

* Remove RewardPool address and rename ValidatorSet address to EpochManager address

* Remove burn contract flag

* Remove non-mintable configuration in genesis

* Lint fixes

* Minor rename

* Update e2e-polybft/e2e/consensus_test.go

Co-authored-by: Stefan Negovanović <93934272+Stefan-Ethernal@users.noreply.github.com>

* Fix comment

* bridge commands relocation (#19)

* bridge commands relocation

* mint command relocation

* un/stake and whitelist from sidechain to validator folder

* sidechain commands relocation

* commands relocation

* deleted license

* build fix

* Build fix

* segmentation fault fix

* added nativeERC20 approve

* command fix

---------

Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>

* Remove IsMintable native token flag (#22)

* Remove IsMintable native token flag

* Remove key from PremineInfo

* Add blade admin flag (#24)

* Introduce blade admin

* Move to the newest main branch on contracts

* Remove NativeTokenConfig from e2e tests

* Comments fix

---------

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>

* UT fix (#25)

* Add default premine for validators (#26)

* Fix consensus `e2e` tests (#28)

* Fix e2e tests

* Fix migration e2e test (#30)

* Remove StakeManagerAddr, StakeTokenAddr and CustomSupernetManagerAddr from bridge config

* Fix integration test (#31)

* Fix fund command UT

* Small fixes

---------

Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>

* Small fixes for e2e tests

* Move and modify `ChangeVotingPower` `e2e` test (#34)

* Move and modify test

* variable name change

---------

Co-authored-by: Dusan <dusan.nosovic@ethernal.tech>

* Fix bridge e2e tests (#33)

* Spin up bridge in bridge e2e tests

* Remove unused param

* Check if bridge is enabled when creating state sync relayer

* Remove bridge non native ERC20 token test case

* Logs

* Fix segmentation fault

* Remove logs

* small fix

* small fix

* Deploy root erc20 token to the rootchain

* Initialize CheckpointManager

* Reduce default stake amount

* TestE2E_Bridge_Transfers_AccessLists fix

* Minor fixes

* Fix TestE2E_AddressLists_Bridge

* Fix TestE2E_Bridge_ERC721Transfer and TestE2E_Bridge_ERC1155Transfer

* Fix TestE2E_Bridge_RootchainTokensTransfers

* Premine fix

* Reduce DefaultStake

---------

Co-authored-by: Goran Rojovic <goran.rojovic@ethernal.tech>

* Fix local deployment scripts (#40)

* Fix cluster script

* Fix docker local deployment

* Rename (rootchain to bridge in command outputs)

* Minor change

---------

Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>

* test fix

* Remove unnecessary forks (#42)

* Remove leftover

* Use in-house built event tracker (#41)

* Use ethernal's event tracker

* Remove log

* EventTracker config in one struct

* Comments fix

---------

Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>

* baseFeeConfig as condition for london fork (#43)

* baseFeeConfig as condition for london fork

* lint fix

* comment fix

* Update command/genesis/polybft_params.go

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>

* deleted unnecessary if

* e2e test fix

* Linter fix

* Add base-fee-config to e2e tests where needed

* Add SetBaseFeeConfig to the legacy e2e framework

* Remove leftover

* Provide non-validator account as receiver of native tokens in TestE2E_Consensus_MintableERC20NativeToken

---------

Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>
Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>

---------

Co-authored-by: Stefan Negovanović <93934272+Stefan-Ethernal@users.noreply.github.com>
Co-authored-by: Stefan Negovanović <stefan@ethernal.tech>
Co-authored-by: Goran Rojovic <goran.rojovic@ethernal.tech>
Co-authored-by: Goran Rojovic <100121253+goran-ethernal@users.noreply.github.com>
bane added a commit that referenced this pull request Mar 14, 2024
* Deploy Network

* Add Permissions

* Minor changes

* Concurrency change

* Destroy Network

* Minor changes

* Minor changes

* Fix commit sha

* Minor changes

* Add Tests

* Disable Tests temporarily

* Nightly Build

* Minor fixes

* Big changes

* Minor changes

* Minor changes

* Minor changes

* Notification fixes
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.

3 participants