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

Refactoring - rebased v8.3.6.0 #1208

Open
wants to merge 6 commits into
base: prestashop/8.x
Choose a base branch
from
Open

Conversation

L3RAZ
Copy link
Collaborator

@L3RAZ L3RAZ commented Mar 4, 2024

No description provided.

btafforeau and others added 3 commits March 4, 2024 17:00
* Modified payment client and configuration builder

* Changed PayPalClient name

* Added builder interface

* CS fix
* Replace client token by user id token

* Implement payment method token

* Interfaces

* PAYSHIP-2631 PayPal Create Order Request DTO

* Added correct class import

* [PAYSHIP-2637] CreatePayPalOrderResponse DTO (#1189)

* Added CreatePayPalOrderResponse DTO

* Added DTOs for create order response

* CS fix

* Added licenses

* [PAYSHIP-2632] Order create refactoring (#1183)

* Created required classes

* Added order create command handler logic

* Added paypal order query handler

* Moved QueryResult to Query namespace

* Added create command and get order query to command bus factory

* CS fix

* PHPStan fixes

* Reverted to old create handler

* PHPStan fixes

* Fixed regex and wrong customerId type

---------

Co-authored-by: Laurynas <[email protected]>
Co-authored-by: Laurynas Šedys <[email protected]>
@L3RAZ L3RAZ changed the title Refacto/payship 2489 2 Refactoring - rebased v8.3.6.0 Mar 4, 2024
@Matt75 Matt75 force-pushed the release/v8.3.6.0 branch 3 times, most recently from 7262153 to 8331d46 Compare March 5, 2024 09:54
Base automatically changed from release/v8.3.6.0 to prestashop/8.x March 5, 2024 15:57
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