Skip to content
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

pcli: add delegate-many transaction command reading from a CSV file #5002

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

hdevalence
Copy link
Member

Describe your changes

Adds a pcli tx delegate-many command that allows delegations based on a CSV file.

Example CSV file:

penumbravalid10n2srs5257a6nxq9x3fhcsqyc0wh8q2lp7f80qd7wlmzdm6znypse23mmh,0.123
penumbravalid1aglayhqq2ec5yfgx8ljqrd9ypnma5uu46hsd0d77dakgu4nywuyqk29d0m,0.201
penumbravalid1lr6rqawqmwetsr8dvdefrpeh7zuamj2ka9jgmesn2xxfgf49hsys79rz9j,0.256

Transaction: f7d386906f5fabc88a5eb051d31bfd45cb51049a7f92f231ed33be3ef6314151

Checklist before requesting a review

  • I have added guiding text to explain how a reviewer should test these changes.

    • Can check the supplied tx
  • If this code contains consensus-breaking changes, I have added the "consensus-breaking" label. Otherwise, I declare my belief that there are not consensus-breaking changes, for the following reason:

    Client-side changes to pcli only.

Example CSV file:

```
penumbravalid10n2srs5257a6nxq9x3fhcsqyc0wh8q2lp7f80qd7wlmzdm6znypse23mmh,0.123
penumbravalid1aglayhqq2ec5yfgx8ljqrd9ypnma5uu46hsd0d77dakgu4nywuyqk29d0m,0.201
penumbravalid1lr6rqawqmwetsr8dvdefrpeh7zuamj2ka9jgmesn2xxfgf49hsys79rz9j,0.256
```

Transaction: f7d386906f5fabc88a5eb051d31bfd45cb51049a7f92f231ed33be3ef6314151
@erwanor erwanor merged commit 4cae498 into main Jan 24, 2025
15 checks passed
@erwanor erwanor deleted the pcli-tx-delegate-many branch January 24, 2025 19:32
@conorsch conorsch mentioned this pull request Jan 29, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants