Flying-Toast/yugo

Add additional mailbox manipulation commands (MOVE, LIST, etc)

Closed this issue · 2 comments

bkono commented

Love the library, nicely done.

Its current focus is clearly around read-only, receiving of messages. I've actually forked it and started adding some additional functionality around LIST and MOVE specifically, and will need to add fetching of prior messages and creation of mailboxes/folders as well.

Are you open to PRs adding additional, non-receiving, functionality? Wanted to check-in on your goals with the lib first before investing in getting my early spikes to PR ready. If yes, any specific requests or contributor guidelines besides the norm (doc comments, tests etc)?

Hi!

My initial use case for the library was indeed very read-only, but I’d be more than happy to review a PR for additional features to make the library more complete.

Aside from the norm, all I’d say is to try and keep the details of IMAP at least somewhat abstracted away so that users need not be familiar with the intricacies of the IMAP spec in order to use the library.

Thanks!

bkono commented

Awesome, so far I've added LIST, MOVE, and CREATE with tests, as well as some smaller related updates (slightly improved IDLE handling, flowing seqnum through the message to support a very easy MOVE workflow, etc). Keeping the API surface area reasonably abstracted from IMAP, since I don't like having to remember much of it either 😆. Will finish confirming they are working as expected, and then split up into a few PRs to keep it from being overwhelming.