feat: Edit Collection & NFT UI #11171
Conversation
✅ Deploy Preview for koda-canary ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
|
||
| const args = [ | ||
| api.tx.nfts.setCollectionMetadata(collectionId, metadata), | ||
| api.tx.nfts.setCollectionMaxSupply(collectionId, item.collection.max), |
There was a problem hiding this comment.
if supply does not change, I would skip the call
|
Amazing ❤️❤️ also I would make the modal wider WDYT @exezbcz |
- added <LogoField /> component to reuse logo field section
…es with "UpdateCollectionStatemine".
hassnian
requested review from
preschian and
vikiival
and removed request for
a team
|
|
will editing an nft create on chain event that will be in the activity tab? |
| const metadata = (await processSingleMetadata( | ||
| collectionMetadata, | ||
| )) as TokenMetadata | ||
| const metadata = await processSingleMetadata<TokenMetadata>(collectionMetadata) |
There was a problem hiding this comment.
update types? or using another types
There was a problem hiding this comment.
the type did not change just using generic types
There was a problem hiding this comment.
metadata.banner does not exists in this type, no?
There was a problem hiding this comment.
oh, yes this type doesn't have banner, should we update the hyperdata type or create a local one
There was a problem hiding this comment.
Rather make PR to kodadot/packages
atm no, the indexer does not have an UPDATE_METADATA event. 
|
PR Type
Context
not closing some issues since some UI/UX changes may be needed
Screenshot 📸
Edit NFT
Edit Collection
Refresh Collection Nfts Metadata