Skip to content

Update webpack configuration to respect Plone excludes URL#848

Open
rodfersou wants to merge 1 commit intomasterfrom
update-webpack-configuration
Open

Update webpack configuration to respect Plone excludes URL#848
rodfersou wants to merge 1 commit intomasterfrom
update-webpack-configuration

Conversation

@rodfersou
Copy link
Copy Markdown
Member

Depends on https://github.com/simplesconsultoria/sc.recipe.staticresources/pull/56/files

And a new release of sc.recipe.staticresources

@rodfersou rodfersou force-pushed the update-webpack-configuration branch from c9643c9 to c0e6679 Compare December 31, 2018 00:39
cursor: pointer;
// FIXME - https://github.com/simplesconsultoria/sc.recipe.staticresources/issues/44
background-image: url('./img/pb_close.png');
background-image: url('++resource++plone.app.jquerytools.pb_close.png');
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member

@hvelarde hvelarde left a comment

Choose a reason for hiding this comment

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

seems something is still not ready with this pull request; seems you need to run webpack to copy the template to the right place.

do you need something else?

@rodfersou
Copy link
Copy Markdown
Member Author

@hvelarde this PR depends on merge/fix this first: https://github.com/simplesconsultoria/sc.recipe.staticresources/pull/56/files

and update references in package.json file

height: 30px;
cursor: pointer;
// FIXME - https://github.com/simplesconsultoria/sc.recipe.staticresources/issues/44
background-image: url('./img/pb_close.png');
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You must exclude the image itself.

Did you have to copy another resource or was it just this one?

Comment thread webpack/package.json
"license": "GPL-2.0",
"dependencies": {
"sc-recipe-staticresources": "simplesconsultoria/sc.recipe.staticresources#1.1b5"
"sc-recipe-staticresources": "simplesconsultoria/sc.recipe.staticresources#1.2b0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Fix package namespace and version.

@idgserpro
Copy link
Copy Markdown
Member

@rodfersou now that you've finished collective/sc.recipe.staticresources#56, do you intend to finish this one?

In any IDG package did you need to copy resources from other packages?

@rodfersou
Copy link
Copy Markdown
Member Author

rodfersou commented Oct 5, 2019

@idgserpro unfortunately I don't have spare time next month to work on it.. but basically need to update staticresources version and maybe reference some images outside this package instead of make a copy.

If you want to give it a try, I can review the changes.

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.

3 participants