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

Api refactor product and order related endpoints #11795

Merged

Conversation

AdamKasp
Copy link
Contributor

@AdamKasp AdamKasp commented Sep 2, 2020

Q A
Branch? 1.8
Bug fix? yes
New feature? no
BC breaks? no
Related tickets part of #11250 based on #11791
License MIT

@AdamKasp AdamKasp requested a review from a team as a code owner September 2, 2020 12:57
@probot-autolabeler probot-autolabeler bot added the API APIs related issues and PRs. label Sep 2, 2020
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch from ca700e6 to 7743db2 Compare September 3, 2020 06:22
@AdamKasp AdamKasp changed the title Api refactor product related endpoints Api refactor product and order related endpoints Sep 3, 2020
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch 8 times, most recently from 2f2c7e0 to 89d5a6f Compare September 7, 2020 09:52
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch 3 times, most recently from dc1c548 to 1096d7a Compare September 8, 2020 09:28
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch 2 times, most recently from 6559859 to 25c244f Compare September 8, 2020 12:58
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch from 25c244f to 8af7e9e Compare September 9, 2020 07:19
@AdamKasp AdamKasp force-pushed the api-refactor-product-related-endpoints branch from 8af7e9e to c64233b Compare September 9, 2020 09:14
<attribute name="openapi_context">
<attribute name="summary">Address cart to given location, logged in Customer does not have to provide an email</attribute>
<attribute name="summary">Adds Item to cart</attribute>
Copy link
Member

Choose a reason for hiding this comment

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

why is item capitalized?

</itemOperation>
</itemOperations>

<collectionOperations />
Copy link
Member

Choose a reason for hiding this comment

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

Why have you moved it here?

@lchrusciel lchrusciel merged commit 54864fb into Sylius:1.8 Sep 9, 2020
@lchrusciel
Copy link
Member

Thank you, Adam! 🥇

@AdamKasp AdamKasp deleted the api-refactor-product-related-endpoints branch July 19, 2021 06:20
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants