Skip to content

Conversation

@Ryouzanpaku
Copy link
Contributor

No description provided.

@Ryouzanpaku Ryouzanpaku requested a review from Copilot October 21, 2025 08:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for mapping the CREATE payment state in the Unzer payment SDK. The changes enable proper handling of payments in the "create" state, which occurs when a payment is initiated but not yet completed (e.g., through Paypage).

Key Changes:

  • Added CREATE state to the BasePayment.State enum
  • Updated state mapping logic to support the new state value (ID 6)
  • Added test coverage to verify correct mapping of the CREATE state

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/main/java/com/unzer/payment/BasePayment.java Added CREATE state to the State enum
src/main/java/com/unzer/payment/communication/mapper/ApiToSdkConverter.java Updated state ID range validation and organized imports
src/test/java/com/unzer/payment/business/PaymentTest.java Added test to verify CREATE state mapping for Paypage payments

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@sixer1182 sixer1182 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please check my comment

@Ryouzanpaku Ryouzanpaku merged commit bcf29e2 into main Oct 22, 2025
2 of 5 checks passed
@Ryouzanpaku Ryouzanpaku deleted the CC-2977/map-payment-create-state branch October 22, 2025 08:46
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.

3 participants