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

Error message doesn't include denoms #30

Open
conorsch opened this issue Feb 27, 2023 · 1 comment
Open

Error message doesn't include denoms #30

conorsch opened this issue Feb 27, 2023 · 1 comment
Assignees

Comments

@conorsch
Copy link
Contributor

Over the weekend the wallet ran out of pizza [sic] tokens, and disbursements started failing with this message:

"error fetching notes: requested amount of 10 exceeds total of 0";

To resolve, I compared the denoms included on the service with the output from the bot's pcli view balance and concluded it was pizza that was missing. Removing the denom from the args and bouncing the service resolved.

Ideally, the error message would include the denom. Low-priority, but look to update the message when making other changes. See relevant Discord discussion.

@conorsch conorsch self-assigned this Jul 14, 2023
@conorsch
Copy link
Contributor Author

@aubrika Would love to tackle this one soon, requires changes to the Penumbra asset code. Will hit you up for consult.

@conorsch conorsch moved this to Next in Testnets Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Next
Development

No branches or pull requests

1 participant