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

fix: topping-up-canister Option 2 command #3548

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

wantedbear007
Copy link
Contributor

@wantedbear007 wantedbear007 commented Sep 27, 2024

Existing: dfx cycles top-up AMOUNT CANISTER_ID --network ic
New: dfx cycles top-up CANISTER_ID AMOUNT --network ic

Cycles topup command takes Canister id first then amount. Without this pattern it returns this error.

error: invalid value 'jqylk-byaaa-aaaal-qbymq-cai' for '': Unknown amount specifier: 'i'

Existing: dfx cycles top-up `AMOUNT` `CANISTER_ID` --network ic
New: dfx cycles top-up `CANISTER_ID` `AMOUNT`  --network ic

Cycles topup command takes Canister id first then amount. Without this pattern it returns this error.

error: invalid value 'jqylk-byaaa-aaaal-qbymq-cai' for '<AMOUNT>': Unknown amount specifier: 'i'
@sa-github-api
Copy link

Dear @wantedbear007,

In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.

If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.

— The DFINITY Foundation

@wantedbear007 wantedbear007 changed the title Fix topping-up-canister Option 2 command fix: topping-up-canister Option 2 command Sep 27, 2024
@jessiemongeon1
Copy link
Contributor

@wantedbear007 Please accept the CLA agreement so we can approve this PR. Thanks!

@wantedbear007
Copy link
Contributor Author

@jessiemongeon1 I have accepted CLA agreement can you please re-run CLA Check Ruleset. Thanks !

@jessiemongeon1 jessiemongeon1 merged commit 81c786e into dfinity:master Sep 27, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants