Skip to content

Include status on payment intents#3298

Open
LikeACloud7 wants to merge 1 commit into
SecureBananaLabs:mainfrom
LikeACloud7:codex/payment-intent-status-743
Open

Include status on payment intents#3298
LikeACloud7 wants to merge 1 commit into
SecureBananaLabs:mainfrom
LikeACloud7:codex/payment-intent-status-743

Conversation

@LikeACloud7
Copy link
Copy Markdown

/claim #743

Summary

  • include status: "pending" in newly-created payment intent responses
  • add focused service tests for the payment intent response shape

Fixes #3297
References #743

Validation

  • node --check apps/api/src/services/paymentService.js
  • node --check apps/api/src/tests/paymentService.test.js
  • node --test apps/api/src/tests/paymentService.test.js
  • node --test apps/api/src/tests/*.test.js
  • git diff --check

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.

Payment intent responses should include an initial status

1 participant