idGeneration is MONGO

[mitar]

Is there a reason why idGeneration has to be MONGO? I use random strings everywhere and would like to use it for this collection as well. Could this be part of options?

[mitar]

Would you be open for a pull request for this?

And do you object of adding underscore as a dependency?

[vsivsi]

I'm in the habit of using mongo style ids because fileCollection (my other main package) requires them for external gridFS compatibility, but there's no reason they have to be used here. If you've already done the work, I'll consider a PR, although I can probably add this quickly if you haven't done it already (including docs).

On underscore, I'd prefer to keep deps to a minimum, and given that I'm using CoffeeScript, many (but certainly not all) uses of underscore have an equivalent idiom in CS.

[mitar]

I have not done anything yet. So if you would like to do it quickly, this would be awesome. :-)

[mitar]

I am also using CoffeeScript in my project (https://github.com/peerlibrary/peerlibrary), but underscore still has a place. For example, this line could be written as unless _.isArray ids.

[vsivsi]

Actually its a bit more work than my initial guess because I have tons of method match checks for Meteor.Collection.ObjectID. I need to think on this a bit, it's probably not too bad but definitely more than the three lines I imagined.

On underscore... true, but I've been burned by some incredibly stupid shit in underscore: jashkenas/underscore#1590 (comment)

[mitar]

Actually its a bit more work than my initial guess because I have tons of method match checks for Meteor.Collection.ObjectID. I need to think on this a bit, it's probably not too bad but definitely more than the three lines I imagined.

Yes, you will probably have to create few helper methods and then replace code with them everywhere.

On underscore... true, but I've been burned by some incredibly stupid shit in underscore:

:-)

[vsivsi]

It's getting late, I'll do this in the morning.

[vsivsi]

Implemented in the branch: https://github.com/vsivsi/meteor-job-collection/tree/new-jc-options

[mitar]

Great! Thanks! :-)

[vsivsi]

FYI, This has been merged into the master branch. I intend to publish a new version to Atmosphere later this evening.

[mitar]

Thanks!

[vsivsi]

Feature released in v0.0.14.