-
-
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][UI] Fix styles path for the shop #6235
[Shop][UI] Fix styles path for the shop #6235
Conversation
pjedrzejewski
commented
Sep 28, 2016
Q | A |
---|---|
Bug fix? | yes |
New feature? | no |
BC breaks? | no |
Related tickets | fixes #X, partially #Y, mentioned in #Z |
License | MIT |
@@ -23,7 +23,7 @@ var paths = { | |||
], | |||
sass: [ | |||
'../UiBundle/Resources/private/sass/**', |
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.
Will it work? With Sylius installed as single components (not sylius/sylius
replacing all the other packages), the UiBundle will be at vendor/sylius/ui-bundle/
and ShopBundle at vendor/sylius/shop-bundle/
.
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.
If you use Shop/Admin/Api, which all depend on Core, I think we won't recommend using them as standalone components and maybe we even remove their composer.json and recommend depending on sylius/sylius
, which solves this and many other problems. Thoughts?
@@ -23,7 +23,7 @@ var paths = { | |||
], | |||
sass: [ | |||
'../UiBundle/Resources/private/sass/**', | |||
'ShopBundle/Resources/private/scss/**' | |||
'Resources/private/scss/**' |
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.
Side note: we have sass
folder in UiBundle and scss
here in ShopBundle.
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.
Yeah, I will clean this up when finalizing the UI.
👍 |
…-styles [Shop][UI] Fix styles path for the shop