-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Create extensible Bulk Txn Submit script #13386
base: main
Are you sure you want to change the base?
Conversation
⏱️ 50m total CI duration on this PR
🚨 2 jobs on the last run were significantly faster/slower than expected
|
32aac12
to
9b6e945
Compare
e025332
to
27688fa
Compare
enum DemoCommand { | ||
Submit(Submit), | ||
CreateSampleAddresses(CreateSampleAddresses), | ||
CleanAddresses(CleanAddresses), |
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.
Perhaps FilterAddresses
or SanitizeAddresses
? From the command name, I was imagining this somehow "removed" addresses, such as removing balances from all addresses.
#[derive(Parser, Debug)] | ||
pub struct CleanAddresses { | ||
#[clap(long)] | ||
pub destinations_file: String, |
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.
destinations_file
-> destination_file
This issue is stale because it has been open 45 days with no activity. Remove the |
It can be used for submitting transactions from single account, it can be used for submitting transactions from a pool of worker accounts.
It can be used for submitting list of transactions from a file.
Description
Type of Change
Which Components or Systems Does This Change Impact?
How Has This Been Tested?
Key Areas to Review
Checklist