Skip to content

Commit e3f7083

Browse files
authored
Standardizxe account flags to char 'a' (#5315)
And rename or delete command specific competing flags. These command flags had chars that were not consistent across commands like "confirmations". It's best not to have command specific single letter char shortcuts.
1 parent 4e59dea commit e3f7083

File tree

9 files changed

+9
-33
lines changed

9 files changed

+9
-33
lines changed

ironfish-cli/src/commands/wallet/burn.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,36 +35,31 @@ This will destroy tokens and decrease supply for a given asset.`
3535
static flags = {
3636
...RemoteFlags,
3737
account: Flags.string({
38-
char: 'f',
38+
char: 'a',
3939
description: 'Name of the account to burn from',
4040
}),
4141
fee: IronFlag({
42-
char: 'o',
4342
description: 'The fee amount in IRON',
4443
minimum: 1n,
4544
flagName: 'fee',
4645
}),
4746
feeRate: IronFlag({
48-
char: 'r',
4947
description: 'The fee rate amount in IRON/Kilobyte',
5048
minimum: 1n,
5149
flagName: 'fee rate',
5250
}),
5351
amount: ValueFlag({
54-
char: 'a',
5552
description: 'Amount of coins to burn in the major denomination',
5653
flagName: 'amount',
5754
}),
5855
assetId: Flags.string({
59-
char: 'i',
6056
description: 'Identifier for the asset',
6157
}),
6258
confirm: Flags.boolean({
6359
default: false,
6460
description: 'Confirm without asking',
6561
}),
6662
confirmations: Flags.integer({
67-
char: 'c',
6863
description:
6964
'Minimum number of block confirmations needed to include a note. Set to 0 to include all blocks.',
7065
required: false,

ironfish-cli/src/commands/wallet/chainport/send.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -43,36 +43,30 @@ export class BridgeCommand extends IronfishCommand {
4343
description: 'Wait for the transaction to be confirmed on Ironfish',
4444
}),
4545
account: Flags.string({
46-
char: 'f',
46+
char: 'a',
4747
description: 'Name of the account to send the asset from',
4848
}),
4949
to: Flags.string({
50-
char: 't',
5150
description: 'The Ethereum public address of the recipient',
5251
}),
5352
amount: ValueFlag({
54-
char: 'a',
5553
description: 'The amount of the asset in the major denomination',
5654
flagName: 'amount',
5755
}),
5856
assetId: HexFlag({
59-
char: 'i',
6057
description: 'The identifier for the asset to use when bridging',
6158
}),
6259
fee: IronFlag({
63-
char: 'o',
6460
description: 'The fee amount in IRON',
6561
minimum: 1n,
6662
flagName: 'fee',
6763
}),
6864
feeRate: IronFlag({
69-
char: 'r',
7065
description: 'The fee rate amount in IRON/Kilobyte',
7166
minimum: 1n,
7267
flagName: 'fee rate',
7368
}),
7469
expiration: Flags.integer({
75-
char: 'e',
7670
description:
7771
'The block sequence after which the transaction will be removed from the mempool. Set to 0 for no expiration.',
7872
}),

ironfish-cli/src/commands/wallet/mint.ts

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -41,38 +41,32 @@ This will create tokens and increase supply for a given asset.`
4141
static flags = {
4242
...RemoteFlags,
4343
account: Flags.string({
44-
char: 'f',
44+
char: 'a',
4545
description: 'Name of the account to mint from',
4646
}),
4747
fee: IronFlag({
48-
char: 'o',
4948
description: 'The fee amount in IRON',
5049
minimum: 1n,
5150
flagName: 'fee',
5251
}),
5352
feeRate: IronFlag({
54-
char: 'r',
5553
description: 'The fee rate amount in IRON/Kilobyte',
5654
minimum: 1n,
5755
flagName: 'fee rate',
5856
}),
5957
amount: ValueFlag({
60-
char: 'a',
6158
description: 'Amount of coins to mint in the major denomination',
6259
flagName: 'amount',
6360
}),
6461
assetId: Flags.string({
65-
char: 'i',
6662
description: 'Identifier for the asset',
6763
required: false,
6864
}),
6965
metadata: Flags.string({
70-
char: 'm',
7166
description: 'Metadata for the asset',
7267
required: false,
7368
}),
7469
name: Flags.string({
75-
char: 'n',
7670
description: 'Name for the asset',
7771
required: false,
7872
}),
@@ -81,7 +75,6 @@ This will create tokens and increase supply for a given asset.`
8175
description: 'Confirm without asking',
8276
}),
8377
confirmations: Flags.integer({
84-
char: 'c',
8578
description:
8679
'Minimum number of block confirmations needed to include a note. Set to 0 to include all blocks.',
8780
required: false,

ironfish-cli/src/commands/wallet/multisig/account/participants.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export class MultisigAccountParticipants extends IronfishCommand {
1111
static flags = {
1212
...RemoteFlags,
1313
account: Flags.string({
14-
char: 'f',
14+
char: 'a',
1515
description: 'Name of the account to list group identities for',
1616
}),
1717
}

ironfish-cli/src/commands/wallet/multisig/commitment/aggregate.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ export class CreateSigningPackage extends IronfishCommand {
1414
static flags = {
1515
...RemoteFlags,
1616
account: Flags.string({
17-
char: 'f',
17+
char: 'a',
1818
description: 'Name of the account to use when creating the signing package',
1919
required: false,
2020
}),

ironfish-cli/src/commands/wallet/multisig/commitment/create.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export class CreateSigningCommitmentCommand extends IronfishCommand {
1515
static flags = {
1616
...RemoteFlags,
1717
account: Flags.string({
18-
char: 'f',
18+
char: 'a',
1919
description:
2020
'Name of the account to use for generating the commitment, must be a multisig participant account',
2121
required: false,

ironfish-cli/src/commands/wallet/multisig/signature/aggregate.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export class MultisigSign extends IronfishCommand {
1515
static flags = {
1616
...RemoteFlags,
1717
account: Flags.string({
18-
char: 'f',
18+
char: 'a',
1919
description: 'Name of the account to use when aggregating signature shares',
2020
required: false,
2121
}),

ironfish-cli/src/commands/wallet/multisig/signature/create.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ export class CreateSignatureShareCommand extends IronfishCommand {
1616
static flags = {
1717
...RemoteFlags,
1818
account: Flags.string({
19-
char: 'f',
19+
char: 'a',
2020
description: 'Name of the account from which the signature share will be created',
2121
required: false,
2222
}),

ironfish-cli/src/commands/wallet/send.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -41,32 +41,27 @@ export class Send extends IronfishCommand {
4141
static flags = {
4242
...RemoteFlags,
4343
account: Flags.string({
44-
char: 'f',
44+
char: 'a',
4545
description: 'Name of the account to send money from',
4646
}),
4747
amount: ValueFlag({
48-
char: 'a',
4948
description: 'The amount to send in the major denomination',
5049
flagName: 'amount',
5150
}),
5251
to: Flags.string({
53-
char: 't',
5452
description: 'The public address of the recipient',
5553
}),
5654
fee: IronFlag({
57-
char: 'o',
5855
description: 'The fee amount in IRON',
5956
minimum: 1n,
6057
flagName: 'fee',
6158
}),
6259
feeRate: IronFlag({
63-
char: 'r',
6460
description: 'The fee rate amount in IRON/Kilobyte',
6561
minimum: 1n,
6662
flagName: 'fee rate',
6763
}),
6864
memo: Flags.string({
69-
char: 'm',
7065
description: 'The memo of transaction',
7166
}),
7267
confirm: Flags.boolean({
@@ -83,7 +78,6 @@ export class Send extends IronfishCommand {
8378
'The block sequence after which the transaction will be removed from the mempool. Set to 0 for no expiration.',
8479
}),
8580
confirmations: Flags.integer({
86-
char: 'c',
8781
description:
8882
'Minimum number of block confirmations needed to include a note. Set to 0 to include all blocks.',
8983
required: false,

0 commit comments

Comments
 (0)