Skip to content

feat(setups): add missing fields to PaymentSetupsRequest#620

Merged
armando-rodriguez-cko merged 1 commit into
masterfrom
feat/payment-setups-request-missing-fields
Jul 20, 2026
Merged

feat(setups): add missing fields to PaymentSetupsRequest#620
armando-rodriguez-cko merged 1 commit into
masterfrom
feat/payment-setups-request-missing-fields

Conversation

@armando-rodriguez-cko

Copy link
Copy Markdown
Contributor

What

Add the billingDescriptor, presentmentDetails and terminal fields to PaymentSetupsRequest.

Why

The PaymentSetup schema defines these as writable fields, and they are already present in PaymentSetupsResponse, but they were missing from the request. This left the request unable to set a billing descriptor, presentment details, or terminal details when creating a Payment Setup.

Surfaced by a full request/response audit of the payment setups endpoint against the swagger spec (follow-up to the confirm-response alignment in 7.12.0/7.12.1).

Verification

  • ./gradlew compileJava — passes.
  • The three types already exist under setups.entities (used by the response).

The PaymentSetup schema includes billing_descriptor, presentment_details and
terminal as writable fields. They are present in PaymentSetupsResponse but were
missing from PaymentSetupsRequest. Add them so a Payment Setup can be created
with a billing descriptor, presentment details, and terminal details.
@sonarqubecloud

Copy link
Copy Markdown

@armando-rodriguez-cko
armando-rodriguez-cko merged commit 7214f77 into master Jul 20, 2026
6 checks passed
@armando-rodriguez-cko
armando-rodriguez-cko deleted the feat/payment-setups-request-missing-fields branch July 20, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants