-
Notifications
You must be signed in to change notification settings - Fork 145
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
Create hip-850.md #850
Create hip-850.md #850
Conversation
✅ Deploy Preview for hedera-hips ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Edit headers to pass validation and add note about previous immutability gurantees Signed-off-by: Michael Garber <[email protected]>
Signed-off-by: Michael Garber <[email protected]>
Signed-off-by: Michael Garber <[email protected]>
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 questions
Signed-off-by: Michael Garber <[email protected]>
Signed-off-by: Michael Garber <[email protected]>
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.
Q: A few more suggestions and waiting on some clarifications.
HIP/hip-850.md
Outdated
|
||
## Specification | ||
|
||
This HIP requires an update to the Supply Key mechanism, enabling it to modify the metadata of NFTs while they are held in the treasury account. The NFT data should remain immutable once distributed, ensuring integrity and trust in the NFT ecosystem. |
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.
Can we expand on this section?
Which existing transaction are we tapping into. Is it just TokenUpdate or some other flow. We just want to be explicit here.
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.
@ty-swirldslabs this part could be expanded on
|
||
## How to Teach This | ||
|
||
Educational initiatives will be developed to guide users in leveraging the new Supply Key functionality, particularly focusing on use cases like gaming and event management. |
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.
Protobuf and docs.hedera.com should explicitly be updated to note the presence of a supply key may imply metadata modification for NFTs should they be sent back to the treasury
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.
And, how this interacts with the HIP-657 metadata key.
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.
@ty-swirldslabs can you address these comments in the HIP
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.
Still a few items to address for clarity.
Specifically it would be good to clarify how this works on top of the approved HIP 657 you also proposed 🙌🏾.
After some more internal discussion, it's quite hard to get consensus on the best strategy to enable this feature of only being able to adapt metadata of a NFT that's in the treasury wallet. There's been three different solutions proposed and the next step is to ask the community about the feature and see what they'd prefer out of the paths forward. The options for this feature to be implemented are:
|
So happy to see this functionality coming back for discussion. I can see opinion is divided. It seems natural to me that it is added to the Supply Key to add 'controlled mutability':
Cons
|
Very useful perspective, thanks @Burstall. Some considerations regarding your pros that I'm curious about
All in all we just want to ensure the feature has a consistent feel that scales for devs and users when interacting with HTS |
Good questions.
a) I'm glad you like my use case - I would love to roll that Smart Contract NFT minter/treasury out to add such functionality (right now, all I expose is a burn - this would be a fun enhancement).
|
Thanks, loving these details :)
I love HIPs with community feedback and input, I'm excited to see the views here as it's a good example for others. I'm gonna unblock my review here as the community has made the context more clear. |
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.
@ty-swirldslabs please update the rejected ideas section with the options other than using the supply_key
.
Also a few open comments that need to be resolved and then I'm good to sign off
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.
LG
My concerns are addressed.
Signed-off-by: Michael Garber <[email protected]>
Signed-off-by: Michael Garber <[email protected]>
Signed-off-by: Michael Garber <[email protected]> Co-authored-by: Michael Garber <[email protected]> Signed-off-by: Kim Rader <[email protected]>
Description:
This PR introduces the implementation of the proposed HIP for enhancing the Supply Key functionality in NFT updates within the treasury wallet. The goal is to enable dynamic, secure updates to NFT data, ensuring data immutability post-distribution. This change is essential for supporting evolving use cases in the Hedera ecosystem.
Discussions Link:
Discussion 660