fix: Allow attachments (incl. delivery receipt) at delivery confirmation #25

Merged

shad0w merged 2 commits from claude/issue-9 into master

2026-06-18 22:34:38 +00:00

shad0w commented

2026-06-18 22:31:40 +00:00

Owner

Automated fix by Claude Code for #9.

Closes #9

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

Automated fix by Claude Code for #9. Closes #9 Review, merge, then create a release tag (vX.Y.Z) to deploy.

shad0w added 1 commit 2026-06-18 22:31:41 +00:00

fix(receipt): upsert Receipt record on repeat confirmations with notes 9adc93e54a

Partial-receipt flows call confirmReceipt multiple times. The nested
`create` on the Receipt relation threw a unique-constraint error on the
second call when both confirmations supplied notes, preventing any
delivery from completing and blocking attachment uploads.

Changed to `upsert` so subsequent confirmations update the existing
Receipt row's notes instead of failing.

Adds integration tests covering full receipt, partial receipt, the
upsert scenario (two confirmations each with notes), and permission guards.

Fixes #9