-
-
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
[Shop] Add ability to show static content #5452
[Shop] Add ability to show static content #5452
Conversation
pamil
commented
Jul 6, 2016
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Related tickets | - |
License | MIT |
* @param ClassMetadata $class | ||
* @param string $staticContentPath | ||
*/ | ||
public function __construct(DocumentManagerInterface $dm, ClassMetadata $class, $staticContentPath) |
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.
Why did you abbreviate?
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.
To be consistent with parent implementation of __construct
c997243
to
ded197e
Compare
ded197e
to
e69c777
Compare
- sylius.behat.context.setup.static_content | ||
|
||
- sylius.behat.context.ui.shop.static_content | ||
|
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.
redundant
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.
Srsly? It just separates contexts from filters.
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.
Yup. Indention separates them enough. We don't have such a separation in any other suite.
{% extends '@SyliusShop/layout.html.twig' %} | ||
|
||
{% block content %} | ||
{% createphp resource as='rdf' noautotag %} |
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.
We should not use createphp for new shop. Please remove it.
Thanks Kamil! |
[Shop] Add ability to show static content