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

Extend the orml-tokens pallet so users are able to mint and burn custom assets #333

Closed
annatekl opened this issue Nov 2, 2023 · 3 comments · Fixed by #359
Closed

Extend the orml-tokens pallet so users are able to mint and burn custom assets #333

annatekl opened this issue Nov 2, 2023 · 3 comments · Fixed by #359
Assignees

Comments

@annatekl
Copy link

annatekl commented Nov 2, 2023

To complete the implementation of the new asset type, CurrencyID, within Pendulum, we require the ability for users and issuers to mint and burn custom assets. This ticket focuses on extending the orml-tokens pallet to facilitate the minting and burning of these custom assets, including CurrencyID tokens.

Requirements:

Extend the orml-tokens pallet within the Pendulum repository to support the minting and burning of custom assets, specifically the CurrencyID tokens.

Technical considerations

This extension should be implemented as a new pallet that is tightly coupled to the orml-tokens pallet. It should draw inspiration from FRAME's pallet-assets and only allow users to manage the mint and burn of CurrencyId::Token(u64) assets, not the others.

Deposit for owning an asset

To prevent adversaries from squatting lots of assets for free, we should require a deposit for owning an asset. For this deposit, we 'reserve' some of the user's balance.
This deposit cannot be unreserved until we add the 'destroy' feature (#368) in the future. Note that the deposit reserve needs to be 'transferred' to the new owner when transferring ownership, see here, ie the new owner should then have to reserve some balance and it should be freed from the original owner.

@annatekl
Copy link
Author

annatekl commented Nov 2, 2023

@annatekl
Copy link
Author

it can be completed at a later stage, it is not required for the poc atm

@ebma
Copy link
Member

ebma commented Nov 20, 2023

@vadaynujra and @annatekl please align on the prioritization here. @vadaynujra, this ticket was moved out of 'ready' on purpose.

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