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

Skip to content

Rework behats after checkout and thanks page#11009

Merged
pamil merged 2 commits into
Sylius:masterfrom
AdamKasp:rework-behats-after-checkout-and-thanks-page
Jan 13, 2020
Merged

Rework behats after checkout and thanks page#11009
pamil merged 2 commits into
Sylius:masterfrom
AdamKasp:rework-behats-after-checkout-and-thanks-page

Conversation

@AdamKasp
Copy link
Copy Markdown
Contributor

Q A
Branch? master
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
License MIT

@AdamKasp AdamKasp requested a review from a team as a code owner January 12, 2020 12:26
@AdamKasp AdamKasp force-pushed the rework-behats-after-checkout-and-thanks-page branch 2 times, most recently from 51caa89 to 727571a Compare January 12, 2020 19:34
@GSadee GSadee added DX Issues and PRs aimed at improving Developer eXperience. Shop ShopBundle related issues and PRs. labels Jan 13, 2020
'order_details_in_account' => '[data-test-show-order-in-account]',
'payment_method_page' => '[data-test-payment-method-page]',
'thank_you' => '[data-test-thank-you]',
'create_account_button' => '[data-test-create-an-account]',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Wrong order of keys

@AdamKasp AdamKasp force-pushed the rework-behats-after-checkout-and-thanks-page branch from 727571a to 374531b Compare January 13, 2020 10:10
/**
* {@inheritdoc}
*/
public function getNumberOfItems(): int
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
public function getNumberOfItems(): int
public function getAmountOfItems(): int

@pamil pamil merged commit fd5e1d4 into Sylius:master Jan 13, 2020
@pamil
Copy link
Copy Markdown
Contributor

pamil commented Jan 13, 2020

Thanks, Adam! 🥇

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. Shop ShopBundle related issues and PRs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants