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.

Sign up for GitHub

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

Jump to bottom

entropy balance <address> should take any address #237

Open
frankiebee opened this issue Sep 25, 2024 · 4 comments · May be fixed by #315
Open

entropy balance <address> should take any address #237

frankiebee opened this issue Sep 25, 2024 · 4 comments · May be fixed by #315
Assignees
@rh0delta
Labels
bug Something isn't working

Comments

@frankiebee
Copy link
Collaborator

right now you have to have the address in your config but that command should take any address.
@frankiebee frankiebee added the bug Something isn't working label Sep 25, 2024
@frankiebee frankiebee mentioned this issue Sep 25, 2024
Merged
8 tasks
@frankiebee frankiebee added this to the Feature completeness milestone Oct 15, 2024
@rh0delta
Copy link
Contributor

@frankiebee should we discuss exposing substrate as part of the Entropy class rather than the instance? or were you thinking of another workaround for this?

@rh0delta
Copy link
Contributor

duplicates #115

@mixmix
Copy link
Contributor

mixmix commented Nov 12, 2024

Example:

➜  sdk git:(main) entropy balance 5GpDAn3x37k8Krd9wfsW8Tf6DxWgPJg5e8hXB3dj3ZGtod6h
18,748,620,727 BITS%                                                                                                                                                                                                                                           ➜  sdk git:(main) 
➜  sdk git:(main) rm -rf ~/.config/entropy-cryptography 
➜  sdk git:(main) entropy balance 5GpDAn3x37k8Krd9wfsW8Tf6DxWgPJg5e8hXB3dj3ZGtod6h
file:///home/ENTROPY/cli/dist/cli.js:3736
    throw new Error(`No account with name or address: "${addressOrName}"`);
          ^

Error: No account with name or address: "5GpDAn3x37k8Krd9wfsW8Tf6DxWgPJg5e8hXB3dj3ZGtod6h"
    at loadEntropy (file:///home/ENTROPY/cli/dist/cli.js:3736:11)
    at Command.<anonymous> (file:///home/ENTROPY/cli/dist/cli.js:12091:27)
    at Command.listener [as _actionHandler] (/home/ENTROPY/cli/node_modules/commander/lib/command.js:542:17)
    at /home/ENTROPY/cli/node_modules/commander/lib/command.js:1502:14
    at /home/ENTROPY/cli/node_modules/commander/lib/command.js:1383:33
    at async Command.parseAsync (/home/ENTROPY/cli/node_modules/commander/lib/command.js:1092:5)

Node.js v20.10.0

@mixmix
Copy link
Contributor

mixmix commented Nov 12, 2024

I think this is something we will not support in TUI

@mixmix mixmix mentioned this issue Nov 12, 2024
Closed
@rh0delta rh0delta self-assigned this Nov 22, 2024
@rh0delta rh0delta linked a pull request Nov 22, 2024 that will close this issue
Open
8 tasks
@rh0delta rh0delta linked a pull request Nov 22, 2024 that will close this issue
[NayNay] Balance for any address & transfer (using only substrate api) #315
Open
8 tasks
@frankiebee frankiebee removed this from the Function Completeness milestone Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rh0delta rh0delta

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[NayNay] Balance for any address & transfer (using only substrate api) entropyxyz/cli
3 participants
@mixmix @frankiebee @rh0delta