Skip to content
This repository was archived by the owner on Jul 19, 2021. It is now read-only.

Commit ee43c20

Browse files
authored
Merge pull request #883 from Shopify/fix-liquid-styles
Fix liquid styles not uploading to Shopify
2 parents 5a115cf + f555f82 commit ee43c20

File tree

2 files changed

+6
-10
lines changed

2 files changed

+6
-10
lines changed

packages/slate-tools/tools/asset-server/index.js

+5-1
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,15 @@ module.exports = class DevServer {
6262
_isChunk(key, chunks) {
6363
return (
6464
chunks.filter((chunk) => {
65-
return key.indexOf(chunk.id) > -1;
65+
return key.indexOf(chunk.id) > -1 && !this._isLiquidStyle(key);
6666
}).length > 0
6767
);
6868
}
6969

70+
_isLiquidStyle(key) {
71+
return key.indexOf('styleLiquid.scss.liquid') > -1;
72+
}
73+
7074
_hasAssetChanged(key, asset) {
7175
const oldHash = this.assetHashes[key];
7276
const newHash = this._updateAssetHash(key, asset);

packages/slate-tools/tools/webpack/config/parts/core.js

+1-9
Original file line numberDiff line numberDiff line change
@@ -50,22 +50,14 @@ module.exports = {
5050
{
5151
test: /\.(liquid|json)$/,
5252
exclude: [
53-
new RegExp('assets/styles'),
53+
/(css|scss|sass)\.liquid$/,
5454
...config.get('webpack.commonExcludes'),
5555
],
5656
loader: 'file-loader',
5757
options: {
5858
name: '../[path][name].[ext]',
5959
},
6060
},
61-
{
62-
test: /assets\/static\//,
63-
exclude: /node_modules/,
64-
loader: 'file-loader',
65-
options: {
66-
name: '[name].[ext]',
67-
},
68-
},
6961
{
7062
test: /(css|scss|sass)\.liquid$/,
7163
exclude: config.get('webpack.commonExcludes'),

0 commit comments

Comments
 (0)