Finalize NFT Marketplace

[green-jay]

Legacy PRs:
#137 (merged for testing purposes)

• Add more tests (e.g. do_buy function with mutation checks, other scenarios)
• Address any other pending comments in legacy PRs
• Do full review of pallet-marketplace (ideally multiple peers)
• Rebenchmark on reference machine
• Remove Marketplace call filter in basilisk runtime

[dmoka]

I am going to pick this up as discussed with @lumir-mrkva that we want to utilize auctions soon on Basilisk.

[green-jay]

I am going to pick this up as discussed with @lumir-mrkva that we want to utilize auctions soon on Basilisk.

The status quo of the marketplace chaged since April. So the list should include additionally:

• migrate to the latest version of pallet-nft
• refactor the marketplace to support multiple currencies

[dmoka]

@green-jay , as for pallet-nft, I just updated to the latest version and everything is still functional, so no breaking changes. I guess it is fine, or did you expect some actual code changes in the marketplace pallet (because of some important behavioral change in the pallet-nft)?

[green-jay]

@green-jay , as for pallet-nft, I just updated to the latest version and everything is still functional, so no breaking changes. I guess it is fine, or did you expect some actual code changes in the marketplace pallet (because of some important behavioral change in the pallet-nft)?

as long as it works with latest changes should be fine

[enthusiastmartin]

Completed?

[dmoka]

Completed?

Nope, we froze the work regarding this, as it had low priority. Some work has been done on branch feat/finalize-nft-marketplace, but not complete.

We are planning to add multicurrency support which would require some work. The plan would be to extend the nft marketplace functionality without breaking the existing APIs.