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

Remove address version config #489

Merged
merged 2 commits into from
Dec 1, 2019
Merged

Remove address version config #489

merged 2 commits into from
Dec 1, 2019

Conversation

belane
Copy link
Member

@belane belane commented Nov 28, 2019

Change NEO 3 version address to 53, this will produce that neo address starts with N.

Needs neo-project/neo#1299
Closes neo-project/neo#993 neo-project/neo#1017

@erikzhang
Copy link
Member

We can just remove the field, and it will be set to the default value from Neo.dll.

@shargon
Copy link
Member

shargon commented Nov 28, 2019

If mainnet and testnet have the same value, i am agree with @erikzhang

Copy link
Contributor

@lock9 lock9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HI Belane, only protocol.testnet.json contains Neo 3 information. Other files contain Neo 2 data. Not sure if you should update all the files or only those neo 3 compatible.

@belane belane changed the title Change address version Remove address version config Nov 29, 2019
@erikzhang erikzhang merged commit ed3e580 into neo-project:master Dec 1, 2019
@belane belane deleted the increase-version branch January 15, 2020 22:45
Jim8y pushed a commit to Jim8y/neo-node that referenced this pull request Aug 18, 2021
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.

Wallets are incompatible between NEO3 and NEO2
4 participants