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

feat: Add detailed RPC error messages #4287

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

crStiv
Copy link

@crStiv crStiv commented Jan 5, 2025

Improve RPC Error Messages

Better error messages for RPC operations with clear explanations and troubleshooting steps.

Changes:

  • Added error_messages.rs with standardized messages
  • Updated error handling in RPC client
  • Added tests

Example:
Before: "unable to confirm transaction"
After: "Unable to confirm transaction. Possible reasons: 1. Transaction expired 2. Insufficient funds 3. Network issues"

Fixes #4202

@mergify mergify bot requested a review from a team January 5, 2025 01:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RpcClient method get_account_with_config() ignores config field encoding and always returns data raw/unparsed
1 participant