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

Inconsistent account naming #270

Closed
lock9 opened this issue Mar 20, 2023 · 9 comments · Fixed by #305
Closed

Inconsistent account naming #270

lock9 opened this issue Mar 20, 2023 · 9 comments · Fixed by #305

Comments

@lock9
Copy link
Contributor

lock9 commented Mar 20, 2023

Some commands will depend on the account prefixed with '@' while others don't.

Example:
neoxp contract run GasToken balanceOf @dev -r

neoxp show balance GAS dev

Using @dev on the show balance command won't work:

neoxp show balance GAS @dev                  
System.Exception: @dev account not found.
@cschuchardt88
Copy link
Member

what does contents of your default.neo-express? From what i see you may need to add # instead of @

@lock9
Copy link
Contributor Author

lock9 commented Nov 6, 2023

Hi @cschuchardt88,
The problem is that the command doesn't use expect the '@' prefix (or any prefix).
Try these:
neoxp contract run GasToken balanceOf @genesis -r
neoxp show balance GAS @genesis

This will work:
neoxp show balance GAS genesis

@cschuchardt88
Copy link
Member

if you replace the @ with hash tag (#) it should work, from what i see in code.

@lock9
Copy link
Contributor Author

lock9 commented Nov 6, 2023

It doesn't:
System.Exception: #genesis account not found

@cschuchardt88
Copy link
Member

It only works for non-genesis. If you name the account, use that name with #

@cschuchardt88
Copy link
Member

cschuchardt88 commented Nov 7, 2023

See #279 - wrap anything that has a @ sign, with quotes.

image

@lock9
Copy link
Contributor Author

lock9 commented Nov 7, 2023

Hi @cschuchardt88,
But that is not the point. The point is that account names are inconsistent. It also doesn't work:

neoxp show balances '@genesis'
System.Exception: @genesis account not found.

@cschuchardt88
Copy link
Member

Do # hash tag instead. That's what code says, but I'll change it. Also it still wont work. I'll fix that too in a min.

@cschuchardt88
Copy link
Member

You just going to have to use # hashtag.

Using anything else will break old batch files that have been developed by users.

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 a pull request may close this issue.

2 participants