search

chadxz / imap-simple

Wrapper over node-imap, providing a simpler api for common use cases
MIT License
243 stars 80 forks source link

Feature/extract fetch messages #61

Closed synox closed 3 years ago

synox commented 5 years ago

just a refactoring, provide fetchMessages() in the api and use it in search().

I didn't see a way to modify the search() method signature while keeping it backward compatible.

btw: removing callbacks, using classes, const and arrow functions would be fun. (Node 6.4)

chadxz commented 5 years ago

@synox what is your npm username so i can add you as a contributor? That way you can cut releases.

synox commented 5 years ago

I am 'aravindo'. I never used npm, so I would be happy if you check PRs and do releases.

chadxz commented 5 years ago

@synox I don't really use this library anymore, so it is mostly neglected. I appreciate you are interested in wanting to add features to it to fit your use cases, but I don't really want to spend time working on this since it's not really important to me anymore. I mostly want to make sure you have what you need to make the changes you want.