feat: added command for minting fungible tokens #321
Conversation
src/mint.rs
Outdated
| @@ -267,6 +269,93 @@ pub fn mint_from_uris( | |||
| Ok(()) | |||
| } | |||
|
|
|||
| #[allow(clippy::too_many_arguments)] | |||
There was a problem hiding this comment.
Can you add a MintFungibleArgs struct here to fix the clippy issue and follow the patterns of other commands?
| @@ -852,6 +852,24 @@ pub enum FindSubcommands { | |||
|
|
|||
| #[derive(Debug, StructOpt)] | |||
| pub enum MintSubcommands { | |||
| /// Mint a normal SPL Token from the Token Program | |||
| Fungible { | |||
There was a problem hiding this comment.
What do you think about changing this to Tokens to clarify and maybe the docs comment can be:
/// Mint SPL tokens from an existing mint account using the Token Program.
There was a problem hiding this comment.
I was thinking of following the already built command structure in create
metaboss create fungible
metaboss mint fungible
Would be a bit relatable for users.
There was a problem hiding this comment.
Yeah, I think the problem is that the naming is already inconsistent as "mint" basically means "create" when used for an asset like a NFT but I also have the "create" commands for setting up a fungible mint. I guess Fungible is ok if we change the doc comment to explain.
Support added for fungible token minting. The following screenshot represents the structure of the command.
The amount doesn't need to take account of decimals as the calculation is done inside the processor function.