search

RowinVanAmsterdam / dating-app-expanded

Repo for the Dating App
1 stars 0 forks source link

relocate and rename index.js in static folder to js/main.js #5

Closed cenikk closed 5 years ago

cenikk commented 5 years ago

Consider removing/renaming the index.js static file so there's no confusion which one is being used (feedback I got from Rijk on my repository as well). I believe by doing this you would also have to change the path in tail.ejs. It was hard for me to see immediately which one of the index.js files you were referring to in tail.ejs, and this (in my opinion) why you shouldn't use too many index files (you have one in your main folder now, as well as in you /routes/, /static/ and /view/.

cenikk commented 5 years ago

Oh btw, you could do the same with your index.css (in you static folder) and put it in a folder called 'css' and rename the file to something like 'style.css' (totally up to you tho).

RowinVanAmsterdam commented 5 years ago

Ah I see, did you know you see the path of the (index) file in the vscode tab? So, it's a little bit less confusing :)