Fixes for IVerifyOptions warnings & socket port declaration

datatypevoid / vulgar

A simple and scalable MEAN stack development kit featuring Angular 2 (Router, Http, Forms, Services, Tests, E2E, Coverage, Dev/Prod), Express, MongoDB, Mongoose, Node, PassportJS, Socket.io, Karma, Protractor, Jasmine, Istanbul, SASS Support, TypeScript, TSLint, NG2Lint, Hot Module Replacement, Docco, Gulp, and Webpack by @datatypevoid

MIT License

286 stars 66 forks source link

Fixes for IVerifyOptions warnings & socket port declaration #14

Closed b3kN closed 8 years ago

b3kN commented 8 years ago

Fix(server.conf.ts): Add ternary function to check for valid PORT & SOCKET_PORT in config.json
Feat(config.example.ts): Add example of config for new implementation guidelines
Fix(passport.conf.ts): Change loginMessage & signupMessage to message to clear IVerifyOptions warning

b3kN commented 8 years ago

Credit to @jimmythesaint82 (https://github.com/jimmythesaint82) for assistance in find for fix

datatypevoid commented 8 years ago

Looks good guys @b3kN @jimmythesaint82 ! The only thing I am not going to keep is the example config object. The reason being is that if you generate the stack as intended with the CLI, the config object is generated and configured for you. Thanks again for the help!

datatypevoid commented 8 years ago

In the end though, I decided to just leave the example config.json object. ;)