proposals icon indicating copy to clipboard operation
proposals copied to clipboard

Add Chain Version into NEP description

Open lock9 opened this issue 6 years ago • 4 comments

Hello, @lllwvlvwlll suggested that we add the Chain Version in our NEPs. I think this is a good practice, since if when we update NEO, we may affect NEPs.

lock9 avatar Aug 05 '19 04:08 lock9

We could update past NEPs to inform which networks they refer to. A field: 'protocol' : 2+, 2, 3, 3+

igormcoelho avatar Aug 17 '19 15:08 igormcoelho

We should abolish the NEO2 NEPs when NEO3 MainNet is launched.

erikzhang avatar Aug 18 '19 05:08 erikzhang

I think all existing ones can be migrated Erik... not too much incompatible stuff I guess (haven't seen yet).

igormcoelho avatar Aug 19 '19 04:08 igormcoelho

Hi @erikzhang, but we are keeping both neo2 and neo3 running simultaneously for some time, aren't we? At least during this period of time, we need to describe if the NEPs are compatible. Today we are updating NEP-3 for neo 3 and these changes are not compatible with neo 2, however, neo 2 is the current version in production mode. This may cause issues for new developers.

I believe this is an good opportunity to filter what neps are included in neo3 and which are 'discarded'. @erikzhang do you think there will be negative impacts? Because if there are only positive impacts (even if they are very small), maybe we can put this forward? 😅 It is a demand from our communities, they need to know in order to replicate it into their tools

lock9 avatar Sep 10 '19 17:09 lock9

We have Neo 2 NEPs obsoleted now, so this seems to be solved.

roman-khimov avatar Sep 13 '22 15:09 roman-khimov