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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[API] Product options (with values) creation and index #11136

Merged
merged 8 commits into from
Feb 25, 2020

Conversation

Zales0123
Copy link
Member

@Zales0123 Zales0123 commented Feb 24, 2020

Q A
Branch? api
Bug fix? no
New feature? yes
BC breaks? no
Deprecations? no
Related tickets
License MIT

PR consists of API for creation and indexing product options (with product option values as sub-resources):

Zrzut ekranu 2020-02-25 o 12 36 00

It also contains a PoC of the Behat extension, that does not allow to run a javascript session for API scenarios with multiple tags (like @ui @javascript @api) 馃拑

@Zales0123 Zales0123 added the Feature New feature proposals. label Feb 24, 2020
@Zales0123 Zales0123 requested a review from a team as a code owner February 24, 2020 12:49
@Zales0123 Zales0123 changed the title [API] Product options (with values) creation and index [WIP][API] Product options (with values) creation and index Feb 25, 2020
@Zales0123 Zales0123 changed the title [WIP][API] Product options (with values) creation and index [API] Product options (with values) creation and index Feb 25, 2020
@pamil pamil merged commit 2caa4dd into Sylius:api Feb 25, 2020
@pamil
Copy link
Contributor

pamil commented Feb 25, 2020

Thank you, Mateusz! 馃

<property name="createdAt" writable="false" />
<property name="updatedAt" writable="false" />
<property name="values" readable="true" writable="true">
<subresource resourceClass="Sylius\Component\Product\Model\ProductOptionValue" collection="true" />
Copy link
Contributor

Choose a reason for hiding this comment

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

What about not using subresources for values but IRIs instead?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature proposals.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants