shad0w/pelagia-portal
1
0
Fork 0
Code Issues 10 Pull requests Projects Releases 2 Packages Wiki Activity Actions

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
Conversation 0 Commits 3 Files changed 3 +96 -21
shad0w commented 2026-06-27 18:54:40 +00:00
Owner
Copy link

Automated fix by Claude Code for #140.

Closes #140

Review, merge, then create a release tag (vX.Y.Z) to deploy.

Automated fix by Claude Code for #140. Closes #140 Review, merge, then create a release tag (vX.Y.Z) to deploy.
shad0w added 1 commit 2026-06-27 18:54:40 +00:00
feat(activity): show partial payment amount in PO timeline
All checks were successful
PR checks / checks (pull_request) Successful in 53s
Details
PR checks / integration (pull_request) Successful in 31s
Details
3335977773 
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
shad0w added 1 commit 2026-06-27 19:54:03 +00:00
Merge remote-tracking branch 'origin/master' into pr141-fix
All checks were successful
PR checks / checks (pull_request) Successful in 52s
Details
PR checks / integration (pull_request) Successful in 32s
Details
65be4ef330 
# Conflicts:
#	App/components/po/po-detail.tsx
shad0w added 1 commit 2026-06-27 21:04:29 +00:00
Merge remote-tracking branch 'origin/master' into pr141-fix
All checks were successful
PR checks / checks (pull_request) Successful in 54s
Details
PR checks / integration (pull_request) Successful in 32s
Details
9f681ace89 
# Conflicts:
#	App/components/po/po-detail.tsx
shad0w merged commit 0fdda87381 into master 2026-06-28 01:41:19 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is broken

  
claude-failed

Automated fix attempt failed

  
claude-pr

Claude opened a PR for this

  
claude-queue

Queued for automated Claude fix

  
claude-working

Claude is working on this

  
epic

Crewing epic � tracking issue (A�M)

  
feature

New functionality

  
interactive

Run with Claude interactively/steered � not safe for the unattended watcher

  
portal

Reported via portal Report Issue button

  
triaged

Triage has run on this portal issue (prevents re-triage)

No labels
bug
claude-failed
claude-pr
claude-queue
claude-working
epic
feature
interactive
portal
triaged
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: shad0w/pelagia-portal#141
No description provided.
Delete branch "claude/issue-140"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?