Update ERC-7066: Move to Last Call

[piyush-chittara]

When opening a pull request to submit a new EIP, please use the suggested template: https://github.com/ethereum/EIPs/blob/master/eip-template.md

We have a GitHub bot that automatically merges some PRs. It will merge yours immediately if certain criteria are met:

• The PR edits only existing draft PRs.
• The build passes.
• Your GitHub username or email address is listed in the 'author' header of all affected PRs, inside .
• If matching on email address, the email address is the one publicly listed on your GitHub profile.

[Pandapip1]

@eth-bot rerun

[eip-review-bot]

✅ All reviewers have approved.

[SamWilsn]

Line 64 says:

lock function MUST revert if token is not already locked.

Emphasis mine. This seems backwards?

---

A bit later you write:

ERC-721 _tansfer function [...]

ERC-721 doesn't have a _tansfer function (nor _transfer), so these requirements don't make sense. You may be thinking of the OpenZeppelin implementation, which does have _transfer.

You can't depend on a specific implementation of ERC-721 in a standard, so you should reword this to be more general. Something like:

ERC-721 functions that transfer ownership of a token MUST revert [...]

---

Line 66's use of "setup" is a bit ambiguous. I'd recommend being more explicit.

[Joeysantoro]

In Line 64:

ERC-721 `_transfer` function MUST revert if token is locked. ERC-721 `_transfer` function MUST pass if token is locked and `msg.sender` is `approved` and `locker` both. After ERC-721 `_transfer`, values of `locker` and `approved` MUST be purged.

This is logically inconsistent and could be made clearer by saying something to the effect of:

ERC-721 `_tansfer` function MUST revert if token is locked unless `msg.sender` is both `approved` and `locker`. In the case where `msg.sender` is both approved and locker, the transfer MUST succeed and the approval and locker must be purged.

[piyush-chittara]

Thanks a lot for the reviews @Joeysantoro @SamWilsn! :D
Your pointers are looked into and resolved.

[eip-review-bot]

All Reviewers Have Approved; Performing Automatic Merge...

[github-actions]

The commit 01978d8 (as a parent of bb18f33) contains errors.
Please inspect the Run Summary for details.