Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Resolve createdByGuest on order fixtures#16342

Merged
GSadee merged 1 commit into
Sylius:1.12from
TheMilek:SYL-3542-fix-order-fixtures
Jun 6, 2024
Merged

Resolve createdByGuest on order fixtures#16342
GSadee merged 1 commit into
Sylius:1.12from
TheMilek:SYL-3542-fix-order-fixtures

Conversation

@TheMilek
Copy link
Copy Markdown
Member

@TheMilek TheMilek commented Jun 4, 2024

Q A
Branch? 1.12
Bug fix? a bit
New feature? no
BC breaks? no
License MIT

@TheMilek TheMilek added the DX Issues and PRs aimed at improving Developer eXperience. label Jun 4, 2024
@TheMilek TheMilek requested review from a team as code owners June 4, 2024 11:18
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 4, 2024

Bunnyshell Preview Environment deleted

Available commands:

  • /bns:deploy to redeploy the environment

@TheMilek TheMilek changed the base branch from 1.14 to 1.12 June 4, 2024 12:12
Comment on lines +80 to +82
$customer = $order->getCustomer();

if ($customer->hasUser()) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
$customer = $order->getCustomer();
if ($customer->hasUser()) {
$customer = $order->getCustomer();
if ($customer->hasUser()) {

@GSadee GSadee merged commit 3865509 into Sylius:1.12 Jun 6, 2024
@GSadee
Copy link
Copy Markdown
Member

GSadee commented Jun 6, 2024

Thank you, Kamil! 🥇

@TheMilek TheMilek deleted the SYL-3542-fix-order-fixtures branch June 6, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DX Issues and PRs aimed at improving Developer eXperience.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants