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

Pass custom error messages (like assertion errors) #74

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Pass custom error messages (like assertion errors) #74

wants to merge 1 commit into from

Conversation

fabian-emilius
Copy link

Change Description

API Changes

  • No changes

Documentation Additions

  • No changes

@jlamarr22
Copy link
Contributor

Hi @fabian-emilius -- thanks for submitting this PR.

Can you provide us with a use-case for this change? Since the full, original error message is already available in the cause field, this change could actually cause more confusion to developers since it would be difficult to determine at first glance where the error originated if you're not familiar with errors thrown deeper down the EOS ecosystem.

I'm happy to help you troubleshoot your application or provide you with some examples of how to deal with these errors.

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 this pull request may close these issues.

2 participants