Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bnpl payment integration #145

Merged
merged 12 commits into from
Nov 14, 2023
Merged

Bnpl payment integration #145

merged 12 commits into from
Nov 14, 2023

Conversation

deryacakmak
Copy link
Member

No description provided.

AlicanAkkus
AlicanAkkus previously approved these changes Nov 7, 2023
@@ -202,6 +202,24 @@ public Object createApplePayMerchantSession(ApplePayMerchantSessionCreateRequest
applePayMerchantSessionCreateRequest, Object.class);
}

public BnplPaymentOfferResponse offerBnplPayment(BnplPaymentOfferRequest bnplPaymentOfferRequest) {
Copy link
Member

Choose a reason for hiding this comment

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

retrieveBnplOffers yapalım mı?


@Data
@SuperBuilder
public class ApmPaymentInitRequest {
Copy link
Member

Choose a reason for hiding this comment

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

InitApmPaymentRequest kullanılabilir. bunu silelim

@Data
@SuperBuilder
@EqualsAndHashCode(callSuper = true)
public class BnplPaymentInitRequest extends ApmPaymentInitRequest {
Copy link
Member

Choose a reason for hiding this comment

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

InitApmPaymentRequest den extend edebilir


@Data
@Builder
public class CreatePaymentItemRequest {
Copy link
Member

Choose a reason for hiding this comment

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

bunu da silebiliriz.


@Data
@Builder
public class ConvertedPaymentTxPayout {
Copy link
Member

Choose a reason for hiding this comment

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

bunu da silebiliriz.

import lombok.Data;

@Data
public class BnplPaymentInitResponse {
Copy link
Member

Choose a reason for hiding this comment

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

InitBnplPaymentResponse olarak değiştirelim

import java.time.LocalDateTime;

@Data
public class PaymentTxResponse {
Copy link
Member

Choose a reason for hiding this comment

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

bunu da silebiliriz.



@Test
void init() {
Copy link
Member

Choose a reason for hiding this comment

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

init_bnpl_payment



@Test
void approve() {
Copy link
Member

Choose a reason for hiding this comment

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

approve_bnpl_payment


@Data
@Builder
public class OfferBnplPaymentRequest {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public class OfferBnplPaymentRequest {
public class BnplPaymentOfferRequest {

import java.util.List;

@Data
public class OfferBnplPaymentResponse {
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
public class OfferBnplPaymentResponse {
public class BnplPaymentOfferResponse {

@deryacakmak deryacakmak merged commit 6e280de into master Nov 14, 2023
2 checks passed
@deryacakmak deryacakmak deleted the bnpl-payment-integration branch November 14, 2023 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants