Skip to content
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

RFC: XCM Asset Metadata #50

Draft
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

mrshiposha
Copy link
Contributor

This is a preliminary version of an RFC describing the metadata communication of asset collections and asset instances (i.e., NFTs).

It is by no means final. Instead, it provides a base for a more formal discussion.
Improvements, discussions, and any other input are enormously appreciated!

The preliminary version of the RFC was requested to be posted during the discussion of the Unique Network's XCM NFT proposal.

The RFC is meant to evolve over time.

RFC Summary

The proposed change provides a general way of communicating metadata of assets and asset instances between consensus systems via XCM.
Currently, there is no standard, general, and easy way to do so.
Communicating metadata enables new cross-consensus use cases for NFTs and might simplify the registration of foreign assets (both fungible and nonfungible).

@mrshiposha
Copy link
Contributor Author

@KiChjang, this is the RFC, the possibility of which we spoke of long ago in Matrix. Could you please take a look?
Note, however, that this is a preliminary version.

@franciscoaguirre @xlc Please take a look as well 🙏

@mrshiposha mrshiposha marked this pull request as draft January 17, 2024 16:19
3. Are the proposed metadata format and operations general enough?
4. Could the proposed instruction set be reduced without losing both generality and convenience?
5. How one could easily pay for the execution of the `ModifyMetadata`, `ApproveMetadataModification`, and `NoteMetadataModificationApproval`?
6. How to make the `MetadataMap` bounded? The same question for `MetadataKeys`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the simply way is define max key length, max value length and max number of keys
another ways is not define anything here, but depends on the overall XCM size limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants