fix(demo): use selected endpoint for transaction broadcast - #126
Conversation
|
Hi @Johnosezele, this is ready for review when you have a chance. It implements the selected broadcast endpoint fix discussed in #123. |
|
Not blocking, just something I noticed while reading through: |
|
tACK f91ca62 on Linux. tested locally across send_page_test.dart and blockchain_service_test.dart everything is green and static analysis is clean. verified that broadcasting now resolves through endpointConfigProvider and passes the active EndpointConfig directly to the client factory. |
Summary
EndpointConfigWhy
Wallet sync and fee estimates already honor the server selected in the demo app, but transaction broadcast created a new client from
defaultEndpoints. This meant switching away from an unavailable default server could restore sync while sends still tried the unavailable server.The send flow now resolves the active network's selected endpoint and passes that exact configuration to the broadcast client factory.
Validation
dart format --output=none --set-exit-if-changed bdk_demo/lib bdk_demo/testdart analyze --fatal-infos --fatal-warnings lib test exampleflutter analyzedart testflutter testThe repository-wide format command was also checked. Local Dart 3.12.2 flags the unchanged generated
lib/bdk.dart, while all changed demo files are clean; CI uses Dart 3.10.0.Closes #123