This is fine, but it'd be more idiomatic to use `P.all` https://github.com/petkaantonov/bluebird/blob/master/API.md#promiseallarraydynamicpromise-values---promise

Incidentally, I linked to the `bluebird` promise module. Its nice, fast, more active dev, so might be worth looking at switching to.