-
Notifications
You must be signed in to change notification settings - Fork 19
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
feat: NFT Auctions #168
feat: NFT Auctions #168
Conversation
CannotChangeForbiddenAttribute, | ||
} | ||
|
||
#[pallet::call] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extrinsic header comments are missing all parameters descriptions, which are very important for code readability and anyone implementing FE
Closing this PR in favor of #592 The new PR has isolated English auction and resolved all applicable comments in this PR. TopUp and Candle auctions are left for a future implementation. Here is a summary of the most important TODOs for the future from this PR:
|
Adds NFT auctions with 3 auction types: English, TopUp and Candle
Early tech specification: https://www.notion.so/Auctions-6227f3d237e44baf8980e83eae74dbb7
https://github.com/w3f/Grants-Program/blob/master/applications/subauction.md
This should be merged to master with a call filter.
Weights and benchmarks to be updated at a later point.