[v3] save won't return the saved document

[zaaack]

Looks like create/update have the same problem. Since bluebird is not used anymore, I highly recommend sindresorhus's p-tap package.

[zaaack]

Sorry, after looking into the source deeper, I find out the code could change to something like

const doc = new Doc({ ...attrs })
doc.save()
return doc

but returning the model instance still could be very helpful.

[vadimdemedes]

but returning the model instance still could be very helpful.

Could you explain where it could be useful?

[zaaack]

Actually, the main reason is this breaking my code when upgrading from v2. In v2, I have some code like this, and not working anymore. This might be not a big deal, some document could help.

const post = (new Post({ title: 'Some title' })).save()

https://github.com/vadimdemedes/mongorito/blob/master/lib/mongorito.js#L609

[vadimdemedes]

This is a good point, I think I'd like that too.

[VitaliyLavrenko]

Save method is async process so it return Promise, why it should return something else?