Skip to content

Show direct deposits in transaction views - #1470

Open
Jimbo4350 wants to merge 1 commit into
feature/cip159-account-build-flagsfrom
fix/cip159-deposit-view
Open

Jimbo4350 wants to merge 1 commit into
feature/cip159-account-build-flagsfrom
fix/cip159-deposit-view

Conversation

@Jimbo4350

@Jimbo4350 Jimbo4350 commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

Dijkstra transactions built with --direct-deposit currently omit the recipient and amount from debug transaction view. This renders direct deposits in both JSON and YAML, using the same account and amount format as withdrawals.

Stacked on #1469 and targeting feature/cip159-account-build-flags. The diff contains only the follow-up commit.

Adds JSON/YAML golden tests for two recipients and a separate Herald fragment for #1470 with a single-line description.

Validation on GHC 9.12.4:

  • Executable and both test suites built successfully.
  • 634 selected golden tests passed, including the JSON/YAML transaction-view tests.
  • Fourmolu, HLint, and git diff --check passed.

No live node submission tests were run.

@Jimbo4350
Jimbo4350 marked this pull request as ready for review October 15, 2026 19:06
@Jimbo4350
Jimbo4350 force-pushed the fix/cip159-deposit-view branch 2 times, most recently from beadab8 to d1a6689 Compare October 15, 2026 19:12
@Jimbo4350
Jimbo4350 force-pushed the feature/cip159-account-build-flags branch from 7fc8240 to cea9124 Compare October 15, 2026 19:12
@Jimbo4350 Jimbo4350 changed the title Show direct deposits in transaction views and test multiple recipients Show direct deposits in transaction views Oct 15, 2026

This branch has not been deployed

No deployments
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.

2 participants