You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Logically speaking, receiving an approval to use the NFT has the same meaning as receiving the actual NFT, as far as the contract cares.
This will make the flow much easier for marketplaces that use approval system, instead of a 2 msg system, where you first send approval, and then send the actual msg.
This will reduce complexity by a lot, and reduce number of queries needed, right now on each marketplace listing, you need to query for owner and approvals, this callback will make those queries not needed as you can just pass data with the callback.
We can add the callback to existing approval msg, or create a new message (much like transfer and send).
Same idea can be applied to revoke.
Benefits:
This will simplify the usage of approvals for contracts, and reduce much of the complexity
Because of 1, this will increase the usage of approvals, which in my opinion are more secure then transferring ownership
Doesn't break or block anything, and is backward compatible.
The text was updated successfully, but these errors were encountered:
How about adding
ReceiveApproval
callback?Logically speaking, receiving an approval to use the NFT has the same meaning as receiving the actual NFT, as far as the contract cares.
This will make the flow much easier for marketplaces that use approval system, instead of a 2 msg system, where you first send approval, and then send the actual msg.
This will reduce complexity by a lot, and reduce number of queries needed, right now on each marketplace listing, you need to query for owner and approvals, this callback will make those queries not needed as you can just pass data with the callback.
We can add the callback to existing approval msg, or create a new message (much like transfer and send).
Same idea can be applied to revoke.
Benefits:
The text was updated successfully, but these errors were encountered: