Skip to content

feat: load real transaction history in demo app#102

Open
j-kon wants to merge 7 commits into
bitcoindevkit:mainfrom
j-kon:feat/bdk-demo-real-transaction-history
Open

feat: load real transaction history in demo app#102
j-kon wants to merge 7 commits into
bitcoindevkit:mainfrom
j-kon:feat/bdk-demo-real-transaction-history

Conversation

@j-kon

@j-kon j-kon commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace transaction history sample rows with active-wallet-backed data
  • map BDK wallet transactions into demo-app history rows
  • add a clear no-active-wallet state before loading transaction history
  • keep widget tests lightweight by overriding wallet availability instead of creating real BDK wallets
  • keep test-only fakes under the transaction feature test helpers

Context

Continues the transaction presentation scaffold from #62 by wiring the list/detail flow to active wallet transaction data instead of sample rows.

The production transaction repository still uses the BDK Dart API to read wallet transactions, while widget tests use fake repositories and a lightweight wallet-availability provider.

Verification

  • dart format --output=none --set-exit-if-changed lib test example bdk_demo/lib bdk_demo/test
  • dart analyze --fatal-infos --fatal-warnings lib test example
  • dart test
  • flutter analyze inside bdk_demo
  • flutter test --no-pub test inside bdk_demo

Validation passed:

  • root Dart tests: 22/22 passed
  • bdk_demo Flutter tests: 181 tests passed
  • no analyzer issues

Also confirmed:

  • transactions_list_page_test.dart does not import package:bdk_dart/bdk.dart
  • no widget test creates a real BDK Wallet
  • production transactions_repository.dart still imports and uses BDK

@j-kon j-kon marked this pull request as ready for review June 30, 2026 14:35
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.

1 participant