Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Reconsider message archiving #245

Open
iandunn opened this issue Jun 17, 2015 · 2 comments
Open

Reconsider message archiving #245

iandunn opened this issue Jun 17, 2015 · 2 comments
Labels
enhancement Priority: Medium

Comments

@iandunn
Copy link
Contributor

iandunn commented Jun 17, 2015

Currently SupportFlow archives messages once they're downloaded, rather than deleting them.

One benefit of this approach is that it's safer; if something goes wrong, or if the user's workflow/use case doesn't match our assumptions, then the messages are still available on the server.

One potential downside, though, is that their mailbox could fill up and then be unable to receive new emails. The SF admin would not know that until someone contacted them through an alternate channel to let them know.

One potential solution that wouldn't impact the current approach would be to have SupportFlow check the mailbox size, and warn admins when it reaches 80%, and let them know to clear out the archive folder if they want.

We should also consider whether we should delete messages by default, though. Either way, maybe we should have a filter to control archiving vs deleting.
@Viper007Bond
Copy link
Contributor

Maybe it's better to have archive as the default with delete being available? Especially if a space checker is added.

@iandunn
Copy link
Contributor Author

iandunn commented Jun 17, 2015

Yeah, I like that idea too.

kovshenin added a commit that referenced this issue Jun 26, 2015
@kovshenin
Add a new supportflow_imap_folders filter.
b120f75 
A lot of IMAP accounts are configured to namespace any mailboxes
with `INBOX.` and attempting to move to `ARCHIVE` or creating it
fails silently, causing the e-mails to be fetched over and over
again. With this filter, one is able to change the name of the
archive box.

This is a temporary solution until more user-friendly archive
mailbox picker is in place, and/or #245 is implemented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Priority: Medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Viper007Bond @iandunn