-
Notifications
You must be signed in to change notification settings - Fork 55
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
soroban-cli: error messages saying to use config identity fund suggest a broken command #1014
Labels
bug
Something isn't working
Comments
willemneal
added a commit
to AhaLabs/stellar-cli
that referenced
this issue
Oct 11, 2023
tsachiherman
added a commit
that referenced
this issue
Oct 16, 2023
fixes #1014 Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
willemneal
added a commit
to AhaLabs/stellar-cli
that referenced
this issue
Oct 20, 2023
fixes stellar#1014 Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
willemneal
added a commit
to AhaLabs/stellar-cli
that referenced
this issue
Oct 20, 2023
fixes stellar#1014 Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
tsachiherman
added a commit
that referenced
this issue
Oct 20, 2023
* Feat!(CLI): remove sandbox (#997) * feat!: remove run_in_sandbox * fix: reorganize tests that rely on network and ignore outside of go test * fix: use feature instead of cfg so that cargo test --list can find tests * fix: move common tests to single functions to skip setup time * feat!: skip wasm in invoke * fix: fully remove all sandbox related types and args * fix: clippy --all (#1013) * fix: clippy and git-hooks * fix: split CLI tests and normal go tests so they can run concurrently * fix(CLI): fund command now can accept a public strkey (#1020) fixes #1014 Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com> --------- Co-authored-by: Tsachi Herman <24438559+tsachiherman@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version are you using?
20.0.0-rc3
What did you do?
What did you expect to see?
❯ soroban config identity fund GDVYX75NCSGCVPVE6LS43DGK643NKJRETSFPXHSYCQIAMBIYVIAZXOJF --network local
What did you see instead?
Discussion
It seems that the error message thinks it should be possible to fund any G address. However the fund command appears to only accept an identity name, not a G address.
For example, the following command works:
❯ soroban config identity fund me --network local
The immediate fix is probably to fix the error message.
But the better fix would be to "fix" the fund command so it lets you fund any G address.
The text was updated successfully, but these errors were encountered: