fix: show own labelled address for swap transactions (closes #55) #57

clawbot · 2026-02-28T17:57:26+01:00

clawbot прокомментировал(а)

2026-02-28 17:57:26 +01:00

Соавтор

In transaction history lists, swap items were showing the contract address or an unlabelled address. This changes both home.js and addressDetail.js to show the user's own labelled wallet address for swap transactions instead.

Closes #55

In transaction history lists, swap items were showing the contract address or an unlabelled address. This changes both `home.js` and `addressDetail.js` to show the user's own labelled wallet address for swap transactions instead. Closes #55

sneak был(а) назначен(а) clawbot

2026-02-28 17:57:26 +01:00

clawbot добавил(а) 1 коммит 2026-02-28 17:57:26 +01:00

fix: show own labelled address for swap transactions in tx lists

check / check (push) Successful in 22s

Информация

2c9a34aff6

For swap transactions in the transaction history list views (home and
addressDetail), display the user's own labelled wallet address instead
of the contract/counterparty address. The contract address is not useful
in the list view — users need to see which of their addresses performed
the swap.

Closes #55

clawbot сослался(ась) на этот запрос на слияние

2026-02-28 17:57:31 +01:00

in the transaction history lists, "swap" items show un-useful address #55

clawbot добавил(а) метку merge-ready

2026-02-28 17:58:48 +01:00

clawbot прокомментировал(а)

2026-02-28 17:58:55 +01:00

Автор

Соавтор

Reviewed. make check passes (14/14 tests, lint clean, formatting clean) and docker build . succeeds.

The logic change is correct: for swap transactions (direction === "contract" + directionLabel === "Swap"), both home.js and addressDetail.js now show tx.from (the user's own wallet) instead of tx.to (the contract/router address). Non-swap contract interactions and regular sent/received transactions are unaffected by the added condition.

Marked merge-ready.

Reviewed. `make check` passes (14/14 tests, lint clean, formatting clean) and `docker build .` succeeds. The logic change is correct: for swap transactions (`direction === "contract"` + `directionLabel === "Swap"`), both `home.js` and `addressDetail.js` now show `tx.from` (the user's own wallet) instead of `tx.to` (the contract/router address). Non-swap contract interactions and regular sent/received transactions are unaffected by the added condition. Marked `merge-ready`.

clawbot сослался(ась) на этот запрос на слияние

2026-02-28 18:01:18 +01:00

fix: show own address for swap transactions in tx list #56

sneak добавил(а) 1 коммит 2026-02-28 18:04:16 +01:00

Merge branch 'main' into fix/55-swap-tx-address

check / check (push) Successful in 22s

Информация

7c53c48cb1

sneak слил(а) коммит 82a7db63b5 в main

2026-02-28 18:16:13 +01:00

sneak удалил(а) ветку fix/55-swap-tx-address

2026-02-28 18:16:13 +01:00