-
Notifications
You must be signed in to change notification settings - Fork 79
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
chore(sdk): Make the "error handler" give more information #777
Conversation
@Chirag-S-Kotian there seems to be an issue with the unit tests, I'll look at them once they're fixed, ok? |
Okay I will implement that and if there is an error in unit test please let me know. |
…eError with improved error messaging
hy @alainncls help me to run the tests locally using pnpm ,tell me the exact command to test the errorhandler. |
If you want to run all unit tests for the SDK: Hope that helps! |
@Nmhth17 review 😌 please . |
What does this PR do?
This PR refactors the error handling mechanism in the SDK to handle various error scenarios more effectively. It improves the clarity and specificity of error messages returned when a transaction fails, especially when dealing with contract function revert errors.
Related ticket
Fixes #769
Type of change
Check list