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

ACP-3558 Merchant payment expense support. #17

Merged

Conversation

abitskil
Copy link
Contributor

@abitskil abitskil commented Aug 16, 2024

Release Table

Module Release Type Constraint Updates
AppPayment major

Module AppPayment

Change log

Breaking Changes

  • Extended the entity support for the merchant transfer operation (payout/reverse payout), so now they support order items and order expenses.
  • Removed PaymentTransmissionsRequest.orderItems transfer field.
  • Removed PaymentTransmission.orderItems transfer field.
  • Adjusted AppPaymentFacade::transferPayments() to process the payment transmission items instead of just an order items.
  • Impacted AppPaymentBackendApiRouteProviderPlugin by facade changes.

Improvements

  • Introduced PaymentTransmissionItem transfer.
  • Introduced PaymentTransmissionsRequest.paymentTransmissionItems transfer field.
  • Introduced PaymentTransmission.paymentTransmissionItems transfer field.

@abitskil abitskil changed the title [WIP] ACP-3558: Merchant payment expense support. WIP: ACP-3558 Merchant payment expense support. Aug 16, 2024
Copy link

codecov bot commented Aug 20, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@abitskil abitskil changed the title WIP: ACP-3558 Merchant payment expense support. ACP-3558 Merchant payment expense support. Aug 20, 2024
@matweew matweew merged commit c56ff8a into master Sep 5, 2024
1 of 2 checks passed
@matweew matweew deleted the feature/acp-2642/acp-3621-merchant-payment-expenses-support branch September 5, 2024 15:32
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