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
Changed attribute value creation route #35865
Changed attribute value creation route #35865
Conversation
0ae0d7e
to
f8dc70c
Compare
f8dc70c
to
2e58ab6
Compare
src/Core/Domain/AttributeGroup/ValueObject/AttributeGroupId.php
Outdated
Show resolved
Hide resolved
src/PrestaShopBundle/Resources/config/routing/admin/sell/catalog/attribute.yml
Show resolved
Hide resolved
4f97c7d
to
d79a72c
Compare
@@ -27,13 +27,7 @@ _features: | |||
prefix: /features | |||
|
|||
_attributes: | |||
resource: "attribute.yml" |
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.
As discussed with @jolelievre, we decided to merge attribute.yml
and attribute_group.yml
. Like that, we can handle routes with {attributeGroupId}
parameter and those without it in the same file.
UI tests are green https://github.com/nicosomb/ga.tests.ui.pr/actions/runs/8690654054 |
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.
Thanks @nicosomb
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.
Hello @nicosomb ,
LGTM ✅
Button to add a new value is well displayed on attributes listing page.
Thanks!
Every detail helps: versions, browser/server configuration, specific module/theme, etc. Feel free to add more information below this table.