lykmapipo / mongoose-gridfs

mongoose gridfs on top of new gridfs api
MIT License
91 stars 39 forks source link

pass mongoose connection for allowing multiple connections #9

Closed FelixFurtmayr closed 7 years ago

FelixFurtmayr commented 7 years ago

The pull request "Handle multiple connections and insure mongoose connection" can be deleted with this.

lykmapipo commented 7 years ago

@RAPIDFACTURE thanks for the PR.

FelixFurtmayr commented 7 years ago

Hi lykmapipo,

Thank you very much. Could you also publish the new version on npm?

By the way - according to the note in the readme: "Note!: Ensure mongoose connection before use", should we place an example like the following into the readme?

var connection = mongoose.connect('mongodb://localhost/test');

 connection.once('open', function() {
      // start gridfs
 });

Regards Felix

lykmapipo commented 7 years ago

@RAPIDFACTURE Sure it will be a good guide. Send PR.