-
Notifications
You must be signed in to change notification settings - Fork 13
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
Use cosmos errors for the backend #1471
Comments
Even further, the handler functions of sdks should accept classic error and cosmos error type and transform the classic error to cosmos error automatically (like the querier functions) |
So there is a draft (#1490). We need to decide what errors we want to have and just start using them. I put few example errors. We can create more over the time. Waiting for your commnet. |
After meeting with @krhubert, we decided to implement one new We will also implement new |
As discussed, we're gonna create those errors as needed during development, so closing this issue for now. |
In many places in the sdk backend we use normal errors instead of cosmos errors.
We eventually transform these errors into
ErrInternal
hereSo we should consider using directly cosmos errors for the backend as they provide more context
ErrInsufficientFunds
,ErrInvalidAddress
,ErrUnauthorized
...The text was updated successfully, but these errors were encountered: