fix: Activity should log partial payment amount #141

Merged
shad0w merged 3 commits from claude/issue-140 into master 2026-06-28 01:41:19 +00:00

3 commits

Author SHA1 Message Date
9f681ace89 Merge remote-tracking branch 'origin/master' into pr141-fix
All checks were successful
PR checks / checks (pull_request) Successful in 54s
PR checks / integration (pull_request) Successful in 32s
# Conflicts:
#	App/components/po/po-detail.tsx
2026-06-28 02:34:26 +05:30
65be4ef330 Merge remote-tracking branch 'origin/master' into pr141-fix
All checks were successful
PR checks / checks (pull_request) Successful in 52s
PR checks / integration (pull_request) Successful in 32s
# Conflicts:
#	App/components/po/po-detail.tsx
2026-06-28 01:24:00 +05:30
Claude (auto-fix)
3335977773 feat(activity): show partial payment amount in PO timeline
All checks were successful
PR checks / checks (pull_request) Successful in 53s
PR checks / integration (pull_request) Successful in 31s
The PO Activity timeline rendered every partial payment as the generic
"Partial payment confirmed". markPaid() already persists the instalment
amount on the PARTIAL_PAYMENT_CONFIRMED action's metadata
(metadata.paymentAmount), so surface it: the row now reads
"Partial payment of <amount> confirmed" using the PO's own currency.

Falls back to the plain label when paymentAmount is missing or
non-numeric (older audit rows) so historical POs never render NaN.

Extracted ACTION_LABELS + the new actionLabel() helper into
lib/po-activity.ts so the label logic is unit-testable without pulling
the server-only PoDetail component (and its storage/auth imports) into
jsdom.

Fixes #140
2026-06-28 00:24:28 +05:30