-
Notifications
You must be signed in to change notification settings - Fork 220
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: added command for minting fungible tokens #321
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -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. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What do you think about changing this to /// Mint SPL tokens from an existing mint account using the Token Program. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I was thinking of following the already built command structure in create Would be a bit relatable for users. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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 |
||
/// Path to the mint_authority keypair file | ||
#[structopt(short, long)] | ||
keypair: Option<String>, | ||
|
||
/// Token Mint address | ||
#[structopt(short = "M", long)] | ||
mint_address: String, | ||
|
||
/// Receiving address, if different from mint authority. | ||
#[structopt(short = "R", long)] | ||
receiver: Option<String>, | ||
|
||
/// Amount of tokens to mint | ||
#[structopt(short, long)] | ||
amount: u64, | ||
}, | ||
/// Mint an asset from the new Token Metadata Program unified handlers. | ||
Asset { | ||
/// Path to the update_authority keypair file | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a
MintFungibleArgs
struct here to fix the clippy issue and follow the patterns of other commands?