darkterra / mongo-scheduler

Persistent event schedule for node.js using mongo as storage
MIT License
8 stars 8 forks source link

fixing scheduler.remove() #7

Closed AlexanderProd closed 5 years ago

AlexanderProd commented 5 years ago

There's a bug that hinders scheduler.remove() from working.

if (typeof name === 'string') {
   query.event = name;
}

needs to be

if (typeof name === 'string') {
   query.name = name;
}
darkterra commented 5 years ago

Well seen ! Alas, I have not had time to add Unit Tests to this feature yet

AlexanderProd commented 5 years ago

@darkterra Can you please publish a new version with the fix. :)

darkterra commented 5 years ago

@AlexanderProd, did you try the version 2.1.2 ?

AlexanderProd commented 5 years ago

Oh sorry didn’t realize you already released a new version on rpm since on it GitHub the latest one is 2.1.1.

darkterra commented 5 years ago

:)