sneak/AutistMask
1
0
Fork 0
Code Issues 66 Pull Requests 3 Actions Packages Projects Releases Wiki Activity

fix: preserve multiple token transfers per tx hash in transaction history #75

Closed
clawbot wants to merge 1 commits from fix/issue-72-address-token-tx-history into main
pull from: fix/issue-72-address-token-tx-history
merge into: :main
Conversation 1 Commits 1 Files Changed 1
+13 -4
clawbot commented 2026-02-28 21:08:47 +01:00
Collaborator
Copy Link
Copy Source

When a swap produces multiple ERC-20 token transfers with the same tx hash (e.g. send WETH + receive USDC), only the last transfer was kept because txsByHash was keyed by bare tx hash. This caused the address-token view to show no transactions for tokens obtained via swap.

Use a composite key (hash:contractAddress) so all token transfers are preserved. The bare-hash normal-tx entry is removed when token transfers replace it to avoid duplication.

docker build . passes.

Closes #72

When a swap produces multiple ERC-20 token transfers with the same tx hash (e.g. send WETH + receive USDC), only the last transfer was kept because `txsByHash` was keyed by bare tx hash. This caused the address-token view to show no transactions for tokens obtained via swap. Use a composite key (`hash:contractAddress`) so all token transfers are preserved. The bare-hash normal-tx entry is removed when token transfers replace it to avoid duplication. `docker build .` passes. Closes #72
clawbot added 1 commit 2026-02-28 21:08:47 +01:00
fix: preserve multiple token transfers per tx hash in transaction history
check / check (push) Successful in 22s
Details
256acbeeec 
When a swap produces multiple ERC-20 token transfers with the same tx
hash (e.g. send WETH + receive USDC), only the last transfer was kept
because txsByHash was keyed by bare tx hash. This caused the
address-token view to show no transactions for tokens obtained via swap.

Use a composite key (hash:contractAddress) so all token transfers are
preserved. The bare-hash normal-tx entry is removed when token transfers
replace it to avoid duplication.

Closes #72
clawbot added the needs-review label 2026-02-28 21:08:58 +01:00
clawbot self-assigned this 2026-02-28 21:08:59 +01:00
clawbot removed their assignment 2026-02-28 21:09:05 +01:00
clawbot commented 2026-02-28 21:09:08 +01:00
Author
Collaborator
Copy Link
Copy Source

Closing as duplicate of #76.

Closing as duplicate of #76.
clawbot closed this pull request 2026-02-28 21:09:09 +01:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No labels needs-review
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
clawbot sneak (Jeffrey Paul)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/AutistMask#75