-
-
Notifications
You must be signed in to change notification settings - Fork 365
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: 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. |
Amazing ❤️❤️ also I would make the modal wider WDYT @exezbcz |
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@kodadot1/[email protected] |
@@ -241,7 +241,7 @@ export interface ActionMintCollection { | |||
|
|||
export enum Collections { |
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.
Why did you remove the set_max?
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.
UPDATE_COLLECTION
is the replacement for that action, or you mean it we should keep that and used it if the user only changes the max supply
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.
yup I would keep it
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.
what about something like this 492f13b since this is the only place where that would be used, alternatively I can just a the old action SET_MAX_SUPPLY
and just call that
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.
Some small things for clarification
pls resolve conflicts |
…nft-gallery into feature/init/swapLanding
blocker #11187 edit: fixed ty: preschian |
Quality Gate passedIssues Measures |
PR Type
Context
not closing some issues since some UI/UX changes may be needed
Screenshot 📸
Edit NFT
Edit Collection
Refresh Collection Nfts Metadata