Skip to content
This repository has been archived by the owner on Dec 7, 2023. It is now read-only.

Add new testnet network id #852

Merged
merged 1 commit into from
Mar 29, 2022
Merged

Add new testnet network id #852

merged 1 commit into from
Mar 29, 2022

Conversation

superboyiii
Copy link
Member

Since testnet data is incompatible now becasue of the modification of native contract, we could create a new testnet N3T5. Wait for a better solution on native contract update.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 2051063427

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 14.936%

Totals Coverage Status
Change from base Build 2050656831: 0.0%
Covered Lines: 107
Relevant Lines: 694

💛 - Coveralls

@Jim8y
Copy link
Contributor

Jim8y commented Mar 29, 2022

Just curious, does the id has any special meaning? or its just randomly picked?

@superboyiii
Copy link
Member Author

superboyiii commented Mar 29, 2022

Just curious, does the id has any special meaning? or its just randomly picked?

BigInteger -> Hex -> UTF8 string -> N3T5

@Jim8y
Copy link
Contributor

Jim8y commented Mar 29, 2022

BigInteger -> Hex -> UTF8 string -> N3T5
LOOOOOOL, Impressive~~~~~way more complex than i thought

@superboyiii superboyiii merged commit 2ad44ef into develop Mar 29, 2022
@superboyiii superboyiii deleted the N3T5 branch March 29, 2022 13:49
erikzhang pushed a commit that referenced this pull request Apr 19, 2022
* neo-cli: tune MaxTransactionsPerBlock for testnet (#850)

* Add new testnet network id (#852)

* typo, comment (#847)

* Install plugin and dependencies. (#807)
roman-khimov added a commit to nspcc-dev/neo-go that referenced this pull request May 3, 2022
roman-khimov added a commit to nspcc-dev/neo-go that referenced this pull request May 4, 2022
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.

None yet

4 participants