Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow account-level operators to set approval for tokens #526

Closed
alexytsu opened this issue Nov 17, 2022 · 1 comment
Closed

Allow account-level operators to set approval for tokens #526

alexytsu opened this issue Nov 17, 2022 · 1 comment

Comments

@alexytsu
Copy link
Contributor

  Re approval semantics, ERC-721 `approve` says:

Throws unless msg.sender is the current NFT owner, or an authorized operator of the current owner.

This suggests that an address that was nominated via SetApprovalForAll can approve other addresses for specific tokens. But it can't call SetApprovalForAll, and neither can an address approved for a single token approve some other address for that token.

So, should we follow the prior art in this case of allowing an account-wide authorised operator to delegate operation of specific tokens?

Originally posted by @anorth in #492 (comment)

@alexytsu
Copy link
Contributor Author

This was created in the wrong repo by accident: filecoin-project/actors-utils#162

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant