Skip to content

Rename the PayPal payment refund command to RefundPayPalPaymentRequest - #829

Merged
GSadee merged 1 commit into
Sylius:phase-2-payment-requestsfrom
rust-le:refactor/refund-paypal-payment-request-rename
Oct 20, 2026
Merged

GSadee merged 1 commit into
Sylius:phase-2-payment-requestsfrom
rust-le:refactor/refund-paypal-payment-request-rename

Conversation

@rust-le

@rust-le rust-le commented Oct 19, 2026

Copy link
Copy Markdown
Contributor
Q A
Branch? phase-2-payment-requests
Bug fix? no
New feature? no
BC breaks? no
Related tickets SY-884, follow-up to #812

RefundPayPalCapturePaymentRequest refunds the whole PayPal payment, while the late capture has its own RefundPayPalLateCapturePaymentRequest, so "Capture" in the name was misleading (#812 (comment)).

The command, its handler and its dispatcher are renamed to RefundPayPalPaymentRequest, RefundPayPalPaymentRequestHandler and RefundPayPalPaymentRequestDispatcher(Interface), with the service ids sylius_paypal.command_handler.refund_paypal_payment and sylius_paypal.dispatcher.refund_paypal_payment_request. Nothing else changes.

@rust-le
rust-le force-pushed the refactor/refund-paypal-payment-request-rename branch from c8878ef to c4fbb86 Compare October 19, 2026 10:40
@GSadee GSadee added the DX label Oct 20, 2026
@rust-le
rust-le force-pushed the refactor/refund-paypal-payment-request-rename branch from c4fbb86 to 6523e7b Compare October 20, 2026 05:56
@GSadee
GSadee merged commit d8a9bc3 into Sylius:phase-2-payment-requests Oct 20, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants