Skip to content
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 imports from root node_modules for scss #356

Merged
merged 8 commits into from
Sep 18, 2020
Merged

Allow imports from root node_modules for scss #356

merged 8 commits into from
Sep 18, 2020

Conversation

GrimLink
Copy link
Contributor

No description provided.

@GrimLink GrimLink changed the title allow imports from root node_modules for scss Allow imports from root node_modules for scss Jun 18, 2019
So We don't have to change the package.json in frontools
Copy link
Contributor

@Igloczek Igloczek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It needs to be optional and configurable, otherwise, it can slow down compiling for everyone.

@GrimLink
Copy link
Contributor Author

@Igloczek I have changed it to a config via the themes.json

@GrimLink GrimLink requested a review from Igloczek September 28, 2019 15:14
@GrimLink GrimLink closed this Oct 4, 2019
@GrimLink GrimLink reopened this Oct 4, 2019
@tdgroot
Copy link

tdgroot commented Oct 11, 2019

+1 on this change.

@GrimLink
Copy link
Contributor Author

@Igloczek I have updated my PR with the latest release.
Any chance you could take a look and maybe add it?

@matthew-muscat
Copy link

+1 on this change here — i've needed this capability from frontools in one of our projects

@GrimLink
Copy link
Contributor Author

Hoi @Igloczek
I noticed another minor release -> releases/tag/1.8.1
Any chance this PR can also be reviewed?

@Igloczek
Copy link
Contributor

Sure, 1.9.0 will probably get those chages if everthing will be fine.

@Igloczek Igloczek merged commit c3d6748 into SnowdogApps:master Sep 18, 2020
Igloczek pushed a commit that referenced this pull request Sep 18, 2020
* allow imports from root node_modules for scss

* IMP: allow node modules from root of Magento 2 install

So We don't have to change the package.json in frontools

* ADD: include path as an option via theme

* FIX: typo in const

* FIX: removed old changes

Whoeps 😅

* IMP: re-add option to scss helper
Igloczek pushed a commit that referenced this pull request Sep 18, 2020
* allow imports from root node_modules for scss

* IMP: allow node modules from root of Magento 2 install

So We don't have to change the package.json in frontools

* ADD: include path as an option via theme

* FIX: typo in const

* FIX: removed old changes

Whoeps 😅

* IMP: re-add option to scss helper
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants