GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
BSD 3-Clause "New" or "Revised" License
22.37k stars 4.05k forks source link

npm run build issue #136

Closed Chronis77 closed 7 years ago

Chronis77 commented 7 years ago

Hi,

I am really hoping someone can point me in the right direction. I need to make a couple of styling and functional changes to the core of grapesjs to cater for my requirements..

Issue number 1: I am running "npm run build". It gets to the webpack command and then fails with a "ERROR in ./dist/grapes.min.js" from UglifyJS Unexpectedtoken: punc()..."

Now i am assuming this has something to do this converting the code to ES5 for Uglify to compile. Any ideas what the issue could be. I haven't really altered the config file and i have installed babel babel-loader and the es2015 present, but it still seems to throw that error.

Issue number 2: If i comment out "webpack.optimize.UglifyJsPlugin" from the plugins it finishes running, however the code i am left with doesn't seem correct. It has multiple references to "__webpack_require__" and if i embed it on the web page i get that error that grapejs is not defined.

My environment is:

The help would be greatly appreciated.

artf commented 7 years ago

Sorry @Chronis77 but can't reproduce it. First of all, did you run npm i? Are you able to start the project with npm start? Can you try to upgrade node to the latest version?

fzs1994 commented 7 years ago

My Environment:

npm start works fine

Chronis77 commented 7 years ago

Thank you so much for the help. It was my node version!

lock[bot] commented 5 years ago

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.