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

Streamline Messages from BlockchainBridge to the Accountant #557

Open
utkarshg6 opened this issue Nov 14, 2024 · 1 comment
Open

Streamline Messages from BlockchainBridge to the Accountant #557

utkarshg6 opened this issue Nov 14, 2024 · 1 comment

Comments

@utkarshg6
Copy link
Collaborator

utkarshg6 commented Nov 14, 2024

The objective is to decrease the total number of messages sent from BlockchainBridge to the Accountant.

Eliminate ScanError messages and integrate their information into existing messages (consider OutboundPaymentsInstructions & QualifiedPayablesMessage). This will streamline communication and reduce message volume.

In certain scenarios, we encounter partial errors that currently trigger ScanError messages. Instead of sending these messages, we propose utilizing the two previously mentioned messages to convey both total and partial failures. By doing this, BlockchainBridge will not segregate the messages, allowing the Accountant to manage success, partial failures, and total failures more effectively.

@utkarshg6 utkarshg6 converted this from a draft issue Nov 14, 2024
@kauri-hero kauri-hero added enhancement New feature or request hardenning and removed enhancement New feature or request labels Nov 17, 2024
@kauri-hero
Copy link
Contributor

Blocked by #456

@kauri-hero kauri-hero moved this from 🆕 New to 🔖 Ready in MASQ Node v2 Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🔖 Ready
Development

No branches or pull requests

2 participants