Greenstand / treetracker-wallet-app

The Greenstand Wallet App
https://wallet.treetracker.org
14 stars 43 forks source link

`npm run build` doesn't work #191

Closed dadiorchen closed 2 years ago

dadiorchen commented 2 years ago
  1. clone the repo
  2. npm ci (I didnt' use npm i, maybe this is the reason, I'm not sure)
  3. npm run build
  10005 root@aliyun:~/treetracker-wallet-web# npm run build
  10006
  10007 > wallet-app@1.0.0 build
  10008 > GENERATE_SOURCEMAP=false react-scripts build
  10009
  10010 Creating an optimized production build...
  10011 Failed to compile.
  10012
  10013 Cannot read properties of undefined (reading 'includes')
  10014
  10015
  10016 root@aliyun:~/treetracker-wallet-web#

dadiorchen commented 2 years ago

@VWRoli seems there are some weird npm things going on currently ;(

@Siddhant-K-code this happens on your side right?

dadiorchen commented 2 years ago

@VWRoli why we need to upgrade to react 17?

Siddhant-K-code commented 2 years ago

Yeah, I was also curious that what is the need for this : node-version >= 16

VWRoli commented 2 years ago

Not working on my side either with my original copy of the repo.

@VWRoli why we need to upgrade to react 17?

We did not upgrade to 17, the project was initialized with 17.

EDIT: I tried to get rid of all the undefined variables in the project, but no luck, still not working. I was thinking can this be related to that we run the mock server in another project @dadiorchen?

EDIT II: npm start still works fine for me

dadiorchen commented 2 years ago

@VWRoli interesting, theoretically, mock server things should have an impact on this

Me too, I can run npm start but can not build, the embarrassing situation is that the command doesn't throw any error message when failing

VWRoli commented 2 years ago

Success