proposals icon indicating copy to clipboard operation
proposals copied to clipboard

New NEPs describing onNEPXXPayment callbacks

Open roman-khimov opened this issue 2 years ago • 7 comments

Please add more implementation references.

roman-khimov avatar Mar 07 '24 15:03 roman-khimov

Seems good to me, maybe we can use NEP-117 and NEP-111, something related to the parent? or we need to be consecutives?

shargon avatar Mar 12 '24 19:03 shargon

or we need to be consecutives?

Huh, we don’t have to. Seems that’s exactly the case of special numbers:

Assign a NEP number (almost always just the next available number, but sometimes it's a special/joke number, like 666 or 3141) in the pull request comments.

AnnaShaleva avatar Mar 12 '24 20:03 AnnaShaleva

I'd go consequently. https://github.com/neo-project/proposals/pull/156 is 24, I'd leave 25 for https://github.com/neo-project/proposals/pull/160 (it came way earlier), so it's 26/27 for these ones.

roman-khimov avatar Mar 12 '24 20:03 roman-khimov

Please add more implementation references.

Burguer neo?

shargon avatar Mar 14 '24 06:03 shargon

Link? We can add everything we have, I think it's pretty popular.

roman-khimov avatar Mar 14 '24 09:03 roman-khimov

Link? We can add everything we have, I think it's pretty popular.

https://github.com/neoburger/code/blob/beb433288bd7b0da9d2245e2a0ae5af5309da7ed/BurgerNEO.cs#L42

shargon avatar Mar 15 '24 13:03 shargon

Added bNEO and also added a note on ASSERTs.

roman-khimov avatar Mar 16 '24 20:03 roman-khimov

@roman-khimov Could you update status and add NEP numbers? No.25 and No.26 will be OK.

superboyiii avatar May 20 '24 09:05 superboyiii

@roman-khimov, the details look good to me, apart for the first motivation paragraph that I will comment there.

On the other hand, I thought there were additional recommendation in the semantics.

In such case I think you can do an append to NEO-11 or NEP-27. This does not necessary need a number. There is no sense to me.

@vncoelho YOU DO NOT CHANGE ANY EXISTING NEP. ITS NEP, A STANDARD, NOT SOME DOCUMENT YOU CAN UPDATE AND UPGRADE FROM TIME TO TIME.

Jim8y avatar May 26 '24 10:05 Jim8y

In my opinion it is a New Informational NEP or a documentation issue.

Do we have any place to classify the NEPs?

For example: https://eips.ethereum.org/informational

vncoelho avatar May 26 '24 11:05 vncoelho

  1. NEP-11/NEP-17 can't be changed. They can only be replaced by some new standards.
  2. Motivational part covers it all, try explaining "I'm implementing the token receiver method for NEP-11/NEP-17 and not NEP-11/NEP-17" quickly. Now you can just say "my contract is NEP-26/NEP-27 compliant".
  3. See also https://github.com/neo-project/proposals/pull/126#discussion_r522770311
  4. There are some details there were not taken into account by original standards, like ASSERT.
  5. There are new opcodes (ABORTMSG/ASSERTMSG) that we know about, but that people will be asking about if not for these standards.

roman-khimov avatar May 26 '24 11:05 roman-khimov

Motivational part covers it all, try explaining "I'm implementing the token receiver method for NEP-11/NEP-17 and not NEP-11/NEP-17" quickly. Now you can just say "my contract is NEP-26/NEP-27 compliant".

Sorry it was not motivational part. It was below it. I already left some comments

vncoelho avatar May 26 '24 11:05 vncoelho

@roman-khimov Please update readme. We can merge this first incase approvals are reverted. @shargon

superboyiii avatar Jun 05 '24 03:06 superboyiii