Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

set CHAIN_ID for Classic #3934

Merged
merged 2 commits into from
Dec 22, 2016
Merged

Conversation

splix
Copy link
Contributor

@splix splix commented Dec 21, 2016

Reads value from params.chainID, if not set uses network_id value.

Related to #3812

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling f79df9f on ethereumproject:splix/chain-id into ** on ethcore:master**.

@jacogr jacogr added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Dec 22, 2016
@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Dec 22, 2016
@gavofyork gavofyork merged commit be75cbf into openethereum:master Dec 22, 2016
@rphmeier rphmeier mentioned this pull request Jan 5, 2017
rphmeier pushed a commit that referenced this pull request Jan 5, 2017
* configurable CHAIN_ID

* set CHAIN_ID for Ethereum Classic
rphmeier pushed a commit that referenced this pull request Jan 5, 2017
* configurable CHAIN_ID

* set CHAIN_ID for Ethereum Classic
rphmeier pushed a commit that referenced this pull request Jan 5, 2017
* configurable CHAIN_ID

* set CHAIN_ID for Ethereum Classic
gavofyork pushed a commit that referenced this pull request Jan 5, 2017
* delay bomb for Classic (ECIP-1010) (#3179)

* delay bomb for classic (ECIP-1010)

* formatting fix after core review, rel [e6b5093]

* Replay protection for Classic

* EIP-155 configuration for Ethereum Classic

* EIP-160 configuration for Ethereum Classic

* Testnet for Ethereum Classic

* revert ETC to correct Network ID

* reuse Morden for Classic Testnet

* set CHAIN_ID for Classic (#3934)

* configurable CHAIN_ID

* set CHAIN_ID for Ethereum Classic

* prettify classic json
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants