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

Fix ProcessPayPalOrderAction #299

Merged
merged 3 commits into from
Jul 18, 2024

Conversation

maikrosenthal
Copy link

Q A
Branch? 1.6 (bug fixes, improvements)
Bug fix? yes
New feature? no

A customer name can contain more than just a first name and last name. It could also contain a salutation, middle names or a title. Using only the first two words of the full_name is therefore not sufficient.

The customer is also added to the address created. This means that the customer can change their shipping address after completing the order and can therefore influence the calculation of shipping costs and tax.

@maikrosenthal maikrosenthal force-pushed the fix_process_paypal_order_action branch from 874ebe0 to 2b4b940 Compare June 19, 2024 11:03
@maikrosenthal maikrosenthal changed the title Fixed process paypal order action Fix ProcessPayPalOrderAction Jun 19, 2024
UPGRADE.md Outdated Show resolved Hide resolved
@maikrosenthal maikrosenthal force-pushed the fix_process_paypal_order_action branch from 2b4b940 to 415eda5 Compare July 18, 2024 06:10
@GSadee GSadee merged commit 907e648 into Sylius:1.6 Jul 18, 2024
14 checks passed
@GSadee
Copy link
Member

GSadee commented Jul 18, 2024

Thanks, @maikrosenthal! 🎉

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.

None yet

2 participants