Skip to content

new setting to delete email from server; don't store duplicate emails;#184

Closed
treworld wants to merge 4 commits intocoddingtonbear:masterfrom
treworld:master
Closed

new setting to delete email from server; don't store duplicate emails;#184
treworld wants to merge 4 commits intocoddingtonbear:masterfrom
treworld:master

Conversation

@treworld
Copy link
Copy Markdown

This will let you choose if you want to delete the messages after they've been imported. A condition has been implemented to prevent duplicate emails from being imported also.

@ad-m
Copy link
Copy Markdown
Collaborator

ad-m commented Oct 2, 2018

@treworld , why do you start adding completely different features in that pull request?

@ad-m
Copy link
Copy Markdown
Collaborator

ad-m commented Oct 2, 2018

@treworld , could you add settings for "don't store duplicate emails;" ? I think that not everyone will be interested in such an action, especially because of performance. I think this can be a new settings per mailbox.

@coddingtonbear
Copy link
Copy Markdown
Owner

@treworld -- just by the way, you can avoid unrelated changes from appearing on a posted pull request if you post your pull request for a branch other than the one you're working on (in this case, probably, master), and continue whatever other feature work you're engaged in in a different branch. It may be counter-intuitive at first, but github PRs update themselves automatically as you push up new changes to the branch you posted the PR for.

@treworld treworld closed this Dec 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants