Auto install dependencies
Hey All,
I think due to the nature of our application we should auto install dependencies for the user's project. That is, we should run npm install and such before we tell the user project generation is complete. I think this will eliminate confusion that users may receive if they do not npm install.
We can also handle any errors that occurred this way.
Let me know what you think!
Hey All,
I think due to the nature of our application we should auto install dependencies for the user's project. That is, we should run npm install and such before we tell the user project generation is complete. I think this will eliminate confusion that users may receive if they do not npm install.
We can also handle any errors that occurred this way.
Let me know what you think!
The first time I tried the extension, I expected it to have installed the dependencies but it wasn't the case, I think it would be a great addition to the extension. create-react-app already does this, so maybe we can use that.
@sahilTara Totally agree with you, when we did the user testing, quite a lot users are expecting to have dependencies installed and they just need to do npm start.
Okay I'll work on this then!
Hey All, I think due to the nature of our application we should auto install dependencies for the user's project. That is, we should run npm install and such before we tell the user project generation is complete. I think this will eliminate confusion that users may receive if they do not npm install. We can also handle any errors that occurred this way. Let me know what you think!
The first time I tried the extension, I expected it to have installed the dependencies but it wasn't the case, I think it would be a great addition to the extension. create-react-app already does this, so maybe we can use that.
Hey @OlivierJM Yeah we actually used create react app to bootstrap a part of the generated code, but I don't think it will help with the dependency loading due to the way the project is structured. Create react app is usually great for 0 pages, but will mess with our multi page structure I think. I think the merging for it will be a bit too dynamic, and will have a tendency to break. If you have any ideas on how you would use CRA or any other suggestions I'm all ears, because I could have overlooked something!