-
Notifications
You must be signed in to change notification settings - Fork 6
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 (pop api): add burn and mint to local fungibles use case #98
Comments
#[pallet::call_index(6)]
pub fn mint(
origin: OriginFor<T>,
id: T::AssetIdParameter,
beneficiary: AccountIdLookupOf<T>,
#[pallet::compact] amount: T::Balance,
) -> DispatchResult
#[pallet::call_index(7)]
pub fn burn(
origin: OriginFor<T>,
id: T::AssetIdParameter,
who: AccountIdLookupOf<T>,
#[pallet::compact] amount: T::Balance,
) -> DispatchResult |
For the native token we won't be able to mint or burn since we are using the DOT token. Minting happens through inflation. In other words, we need Polkadot root origin to do that which we can not :) This is a good find, we will have to look for what error to return ( |
@Daanvdplas But this is specifically in the Pop network case that use DOT as a chain token right? I think POP API still need to support the native token mint when it is used in other Substrate-based blockchain. |
This will still require the root origin most of the time because allowing anyone to mint the native token of the chain seems like a unlikely configuration (difficult to create value for your token if everyone can create them our of thin air). A chain using a contract to mint its native token could be a use case but TMO not reason enough to include it now. In general, the |
Closed with #150 |
Based on the
Mintable
andBurnable
standards.standards:
As you can see they also emit the
Transfer
eventThe text was updated successfully, but these errors were encountered: