coderdojonijmegen / ninja-chat

Other
0 stars 3 forks source link

Make repository deployable on heroku from root directory #10

Closed DaanWillems closed 6 years ago

DaanWillems commented 6 years ago

Versie die gedeployed is met deze aanpassingen: https://ninjachattestdaan.herokuapp.com/ (om te kijken of alles werkt)

bashopman commented 6 years ago

Ziet er goed uit, wat mij betreft. @louiswolf als deze gemerged is, kan de server repo opgeruimd worden. Kun jij dat dan doen?

CoderDojo024 commented 6 years ago

Er lijkt toch nog iets niet goed te gaan..

-----> App not compatible with buildpack: https://buildpack-registry.s3.amazonaws.com/buildpacks/heroku/nodejs.tgz

! ERROR: Application not supported by 'heroku/nodejs' buildpack

!

! The 'heroku/nodejs' buildpack is set on this application, but was

! unable to detect a Node.js codebase.

!

! A Node.js app on Heroku requires a 'package.json' at the root of

! the directory structure.

!

! If you are trying to deploy a Node.js application, ensure that this

! file is present at the top level directory. This directory has the

! following files:

!

! client.zip

! docs/

! LICENSE

! README.md

! src/

!

! If you are trying to deploy an application written in another

! language, you need to change the list of buildpacks set on your

! Heroku app using the 'heroku buildpacks' command.

!

! For more information, refer to the following documentation:

! https://devcenter.heroku.com/articles/buildpacks

! https://devcenter.heroku.com/articles/nodejs-support#activation

   More info: https://devcenter.heroku.com/articles/buildpacks#detection-failure

! Push failed

DaanWillems commented 6 years ago

Er wordt geklaagd in dat bericht dat de package.json niet aanwezig is. Ook wordt er een lijst gegeven van bestanden die wel aanwezig zijn maar deze lijst komt niet overeen met de bestanden die op dit moment in de root directory staan. image

De package.json is wel degelijk aanwezig, weet je zeker dat je de juiste versie hebt geprobeerd te deployen?

CoderDojo024 commented 6 years ago

Ik zat te slapen. Dacht dat je aanpassingen al gemerged waren.