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
Allow writing product options via API #7811
Allow writing product options via API #7811
Conversation
d954c9d
to
6cff941
Compare
@@ -85,6 +93,246 @@ public function it_allows_showing_product_option() | |||
} | |||
|
|||
/** | |||
* @test | |||
*/ | |||
public function it_does_not_allow_delete_product_option_if_it_does_not_exist() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it_does_not_allow_delete
-> it_does_not_allow_to_delete
/** | ||
* @test | ||
*/ | ||
public function it_allows_delete_product_option() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it_allows_delete
-> it_allows_deleting
/it_allows_to_delete
/** | ||
* @test | ||
*/ | ||
public function it_allows_create_product_option_with_multiple_translations() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it_allows_create
-> it_allows_to_create
6cff941
to
c1641f1
Compare
Made the changes suggested - thanks for the quick approval! |
Thank you Andreas, nice work! We will also need documentation, could you submit a separate PR? |
@pjedrzejewski Sure thing, I'd combine it with the docs for #7816 if that's alright with you? |
@alcaeus Sure! |
Allow writing product options via API
This PR replaces #7547. It exposes the
translations
property for product options to the API. It also re-introduces creating, updating and deleting product options via the API, which was removed in #7482.