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

refactor(nft-fractionalization): use asset-ops #11

Draft
wants to merge 2 commits into
base: xcm-nft-dev-env
Choose a base branch
from

Conversation

mrshiposha
Copy link

@mrshiposha mrshiposha commented Sep 3, 2024

This PR introduces the usage of new NFT traits to pallet-nft-fractionalization.

Thanks to the new traits, the pallet can now interact with collection-less NFTs in addition to the in-collection ones (or even fractionalize whole collections if wanted).
Also, since both pallet-uniques and pallet-nfts implement the new traits, they can both be integrated with fractionalization
(Note: for pallet-uniques UpdateMetadata<CanTransfer> is needed, though).

Another change is that fractionalized assets Name and Symbol are now NFT ID-dependent, and this is fully configurable by the Runtime (in contrast to the old implementation where the Name was actually the "name prefix").
This also means that the format of the NFT ID in the fractionalized name is now configurable instead of hard-coded.

Note: the implementation of the new traits for pallet-nfts can be found on this branch: https://github.com/UniqueNetwork/polkadot-sdk/tree/xcm-nft-dev-env

TODO Benchmarks were broken even before the changes. They need to be fixed.
TODO This pallet doesn't support multiple instances in the Runtime. I believe it should be fixed as well.

@mrshiposha mrshiposha force-pushed the feature/pallet-frac-use-asset-ops branch 4 times, most recently from 21c2b30 to 57775bc Compare September 5, 2024 20:27
@mrshiposha mrshiposha force-pushed the feature/pallet-nfts-impl-asset-ops branch from fe88c93 to 20cd25d Compare September 6, 2024 11:22
@mrshiposha mrshiposha force-pushed the feature/pallet-frac-use-asset-ops branch from 57775bc to 3f12aeb Compare September 6, 2024 11:54
@mrshiposha mrshiposha force-pushed the feature/pallet-frac-use-asset-ops branch from 3f12aeb to ddd8d2a Compare October 30, 2024 09:03
@mrshiposha mrshiposha changed the base branch from feature/pallet-nfts-impl-asset-ops to xcm-nft-dev-env October 30, 2024 09:03
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.

1 participant