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

🔧 Better Errors #221

Open
gluax opened this issue Oct 1, 2024 · 0 comments
Open

🔧 Better Errors #221

gluax opened this issue Oct 1, 2024 · 0 comments
Labels
priority: low Should be fixed soon type: chore 🔧 No production code change

Comments

@gluax
Copy link
Collaborator

gluax commented Oct 1, 2024

🔧 Chore

Add in some more error messages for convenience.

Motivation

This is for more easily understandable/readable error messages.

For example, currently, if you try to load a nonexistent DR you just get an error with the bytes of the dr_id and a message "not found" in a JSON blob.

Sometimes, this is enough, but was the DR not found in the timeouts, status, or somewhere else? This information would be helpful for people calling contract methods, likely mostly us for debugging purposes.

However, it's a helpful change and doesn't take much effort.

@gluax gluax added the type: chore 🔧 No production code change label Oct 1, 2024
@mariocao mariocao added the priority: low Should be fixed soon label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: low Should be fixed soon type: chore 🔧 No production code change
Projects
None yet
Development

No branches or pull requests

2 participants