-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[ProductAttribute] Product attributes feature #4846
Conversation
tuka217
commented
Apr 21, 2016
•
edited
edited
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Deprecations? | no |
Related tickets | - |
License | MIT |
|
||
Assert::true( | ||
$this->indexPage->isResourceOnPage(['name' => $name]), | ||
sprintf('The product attribute with name %s should appear on page, but it do not.', $name) |
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'
.
<h4 class="ui dividing header">{{ 'sylius.ui.configuration'|trans }}</h4> | ||
<div class="ui segment configuration"> | ||
{% if form.configuration is defined %} | ||
{% for field in form.configuration %} |
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.
Maybe you could use {% for field in form.configuration if form.configuration is defined %}
?
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.
Unfortunately I can't :(
7f849e4
to
9b0679b
Compare
templates: | ||
form: SyliusAdminBundle:ProductAttribute:_form.html.twig | ||
requirements: | ||
type: text|textarea|checkbox|integer|percent|datetime|date |
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.
I know this will create a bug, but we should not restrict it here, because otherwise we prevent the developer from introducing his own types.
…t attribute [Behat] Implementation of scenarios
9b0679b
to
2a2e1ab
Compare
2a2e1ab
to
7659000
Compare
Nice work Ania, thank you! |