Skip to content

Update app.js#1

Open
jogind3r wants to merge 1 commit into
Createdd:masterfrom
jogind3r:patch-1
Open

Update app.js#1
jogind3r wants to merge 1 commit into
Createdd:masterfrom
jogind3r:patch-1

Conversation

@jogind3r
Copy link
Copy Markdown

@jogind3r jogind3r commented Jan 8, 2018

I am proposing one addition and two changes.
The project was not working and it requires these changes to work properly.
Please review and update.

  1. The app middleware was not calling next(); hence halting the app.
  2. OPTIONS should be cap
  3. and mongo connect should have {useMongoClient: true,}

NOTE: further more i am getting this warning. DeprecationWarning: Mongoose: mpromise (mongoose's default promise library) is deprecated, Ideas!!

I am proposing one addition and two changes.
The project was not working and it requires these changes to work properly.
Please review and update.
1. The app middleware was not calling   next(); hence halting the app.
2. OPTIONS should be cap
3. and mongo connect should have {useMongoClient: true,}

NOTE: further more i am getting this warning. DeprecationWarning: Mongoose: mpromise (mongoose's default promise library) is deprecated, Ideas!!
@lonerz
Copy link
Copy Markdown

lonerz commented Mar 15, 2018

For the mongoose promise, I just did mongoose.Promise = global.Promise;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants