Skip to content

fix(rest/nodejs): include line items in shipped events - #178

Merged
damaz91 merged 1 commit into
Universal-Commerce-Protocol:mainfrom
FanouZeng-TT:fix/node-shipped-event-line-items
Aug 12, 2026
Merged

fix(rest/nodejs): include line items in shipped events#178
damaz91 merged 1 commit into
Universal-Commerce-Protocol:mainfrom
FanouZeng-TT:fix/node-shipped-event-line-items

Conversation

@FanouZeng-TT

Copy link
Copy Markdown
Contributor

Summary

  • Populate Node shipped fulfillment events with the shipped order line item ids and quantities instead of an empty array.
  • Add a fulfillment regression test that completes a checkout, ships its order, and verifies the event line item projection.

Test plan

  • npm --prefix rest/nodejs test
  • npm --prefix rest/nodejs run build
  • uvx --directory . pre-commit run --files rest/nodejs/src/api/checkout.ts rest/nodejs/test/fulfillment.test.ts
  • git diff --check

@damaz91 damaz91 added status:needs-triage Signal that the PR is ready for human triage status:under-review and removed status:needs-triage Signal that the PR is ready for human triage labels Aug 12, 2026
@damaz91
damaz91 merged commit 8e67cef into Universal-Commerce-Protocol:main Aug 12, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants