Skip to content

Conversation

@jogind3r
Copy link

@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

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