Repository navigation
deprecate(payments): deprecate top-level retry max_attempts and end_after_days - #260
armando-rodriguez-cko wants to merge 1 commit into
Conversation
…fter_days The retry object of POST /payments has no max_attempts or end_after_days; they belong to dunning. Payment sessions now share CustomerRetry from common (the old import path still works). Retry and customer retry fields are documented.
🔴 Risk Classification: MAJORApproval route: AI Review + Human Approval Required Classification reasons
Operational gates
Files analysed: 4 wall-e 2026.06.19-02 · policy |
🔬 Debug — why this classification?Each reason code emitted by the classifier, its source clause in the AI in SDLC Control Framework, and what it means.
Kinds:
See issue #3 for the proposal to formalise this map as Appendix A of the standards doc. wall-e 2026.06.19-02 · debug |
🟢 Advisory review: Looks good to meThis PR still needs a human approval — wall-e cannot auto-approve it. For what it's worth, I read the diff and found nothing I'd block on. Deprecates top-level What I checked
This is not an approval. wall-e cannot auto-approve this PR — it is an opinion to help whoever does. Advisory review · us.anthropic.claude-sonnet-4-6 · wall-e 2026.06.19-02 |
|



Summary
The
retryobject ofPOST /paymentshas no top-levelmax_attemptsorend_after_days; they belong todunning. They are now deprecated. Payment sessions now use the sharedCustomerRetry, and the retry and customer retry fields are documented.Changes
checkout_sdk/payments/payments.py:PaymentRetryRequest.max_attemptsandend_after_daysdeprecated in favour ofdunning;PaymentRetryRequest,DunningRetryRequestandDowntimeRetryRequestdocumented from the speccheckout_sdk/common/common.py:CustomerRetrydocumentedcheckout_sdk/payments/sessions/sessions.py: the duplicateCustomerRetryreplaced by the one incommon(the old import path still works)tests/payments/retry_request_serialization_test.py(new): every retry property,enabled=false,max_attempts=0, the deprecated fields, roundtrips and the spec example on all three endpointsAPI Reference
POST /payments(retry): https://api-reference.checkout.com/tag/payments/POST/paymentsPOST /hosted-payments(customer_retry): https://api-reference.checkout.com/tag/hosted-payments-page/POST/hosted-paymentsPOST /payment-links(customer_retry): https://api-reference.checkout.com/tag/payment-links/POST/payment-linksPOST /payment-sessions(customer_retry): https://api-reference.checkout.com/tag/flow/POST/payment-sessionsBreaking changes
None.
PaymentRetryRequest.max_attemptsandend_after_daysare deprecated and will be removed in the next major version.README
Not affected: the README does not document these types.