Express server #10
Express server #10
Conversation
There was a problem hiding this comment.
I like the idea of this change, though it would break the current functionality of http://morriswchris.github.io/proast/ since GitHub pages does not support backend technologies.
Maybe we can try and setup a heroku app prior merging this in? I can help get that setup
| var app = express(); | ||
| var port = process.env.PORT || 3000; | ||
|
|
||
| app.use(express.static(__dirname + '/')); |
There was a problem hiding this comment.
Could you update this to use the path library? CodeClimate flagged with with Use path.join() or path.resolve() instead of + to create paths.. re: http://eslint.org/docs/rules/
There was a problem hiding this comment.
I'll update the PR to include path. I'm not able to deploy a Heroku instance of your project, only my fork. I suggest enabling Automatic Deployments so you don't have to juggle multiple remotes.
|
Finally updated my PR |
Creates an Express server to serve a random curse, per conversation in #1.
Running the server
Install Express
npm installRun with
npm startjs/main.jswill send an AJAX request to/randomwhich will return a randomly chosen curse.