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

Skip to content

[Api] Locale code on checkout#12139

Merged
lchrusciel merged 4 commits into
Sylius:masterfrom
Tomanhez:locale-code-on-checkout
Dec 16, 2020
Merged

[Api] Locale code on checkout#12139
lchrusciel merged 4 commits into
Sylius:masterfrom
Tomanhez:locale-code-on-checkout

Conversation

@Tomanhez
Copy link
Copy Markdown
Contributor

@Tomanhez Tomanhez commented Dec 8, 2020

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

@Tomanhez Tomanhez added Feature New feature proposals. API APIs related issues and PRs. labels Dec 8, 2020
@Tomanhez Tomanhez requested a review from a team as a code owner December 8, 2020 19:42
Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch from 271fff1 to d9e5d66 Compare December 9, 2020 09:58
Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
Comment thread src/Sylius/Component/Core/Model/Channel.php Outdated
Comment thread src/Sylius/Bundle/ApiBundle/CommandHandler/PickupCartHandler.php Outdated
Comment thread features/checkout/seeing_order_summary/seeing_order_locale.feature Outdated
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch 2 times, most recently from d9dca96 to 1e0f8ad Compare December 10, 2020 10:02
Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
Comment thread src/Sylius/Bundle/ApiBundle/CommandHandler/PickupCartHandler.php Outdated
Comment thread src/Sylius/Bundle/ApiBundle/CommandHandler/PickupCartHandler.php Outdated
Comment thread src/Sylius/Bundle/ApiBundle/Resources/config/api_resources/Order.xml Outdated
<group>shop:order:account:read</group>
<group>checkout:read</group>
<group>cart:update</group>
<group>order:read</group>
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.

As you are touching this area, can we introduce shop: prefix to order read serialization? Can be done in separate PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok I do it in new PR after merge this

@lchrusciel
Copy link
Copy Markdown
Contributor

Rebase needed

Comment thread src/Sylius/Behat/Context/Api/Shop/CartContext.php Outdated
Comment thread src/Sylius/Behat/Context/Api/Shop/CartContext.php Outdated
Comment thread src/Sylius/Behat/Context/Api/Shop/CartContext.php Outdated
Comment thread src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php Outdated
Comment thread src/Sylius/Bundle/ApiBundle/CommandHandler/PickupCartHandler.php Outdated
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch from 1e0f8ad to ce46eeb Compare December 11, 2020 10:20
@probot-autolabeler probot-autolabeler Bot added the Maintenance CI configurations, READMEs, releases, etc. label Dec 11, 2020
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch 2 times, most recently from 364322f to dc4fe4d Compare December 11, 2020 20:05
Comment thread features/checkout/seeing_cart_locale.feature Outdated
Comment thread features/checkout/seeing_cart_locale.feature Outdated
Comment thread features/checkout/seeing_cart_locale.feature Outdated
Comment thread features/checkout/seeing_cart_locale.feature Outdated
Comment thread src/Sylius/Bundle/ApiBundle/Resources/config/api_resources/Order.xml Outdated
Comment thread src/Sylius/Bundle/ApiBundle/Resources/config/api_resources/Order.xml Outdated
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch 2 times, most recently from c3f1eff to 78f3415 Compare December 15, 2020 11:26
Comment thread src/Sylius/Bundle/ApiBundle/Resources/config/api_resources/Order.xml Outdated
@Tomanhez Tomanhez force-pushed the locale-code-on-checkout branch from 78f3415 to 1a9a681 Compare December 16, 2020 09:41
@lchrusciel lchrusciel removed the Maintenance CI configurations, READMEs, releases, etc. label Dec 16, 2020
@lchrusciel lchrusciel merged commit 1771cad into Sylius:master Dec 16, 2020
@lchrusciel
Copy link
Copy Markdown
Contributor

Thanks, Tomasz! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API APIs related issues and PRs. Feature New feature proposals.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants