Opened 3 years ago

Closed 14 months ago

#1486686 closed Feature Patches (fixed)

Delete messages instead of moving them to Trash when in Junk folder

Reported by: dennylin93 Owned by: alec
Priority: 5 Milestone: 0.8-rc
Component: User Interface Version: 0.4-beta
Severity: minor Keywords:
Cc: dennylin93@…

Description

Most email solutions delete messages directly when in the Junk folder. There's no need to move them to Trash, then delete them.

I've provided a quick fix that works, but someone might want to redo it.

Attachments (3)

delete-junk.patch (2.2 KB) - added by dennylin93 3 years ago.
delete-junk-configurable.patch (3.7 KB) - added by dennylin93 2 years ago.
delete-junk-user-configurable.patch (7.7 KB) - added by alec 15 months ago.
Updated patch

Download all attachments as: .zip

Change History (13)

Changed 3 years ago by dennylin93

comment:1 Changed 3 years ago by dennylin93

  • Cc dennylin93@… added

comment:2 follow-up: Changed 3 years ago by thomasb

  • Severity changed from normal to minor
  • Type changed from Bugs to Patches

Most email solutions? I just checked Apple Mail, Thunderbird and Postbox and none of them does that. I don't think this should be the default behavior. Keep it as patch.

comment:3 Changed 3 years ago by alec

  • Resolution set to wontfix
  • Status changed from new to closed

-1 from me too.

comment:4 in reply to: ↑ 2 Changed 3 years ago by dennylin93

Replying to thomasb:

Most email solutions? I just checked Apple Mail, Thunderbird and Postbox and none of them does that. I don't think this should be the default behavior. Keep it as patch.

Gmail, Yahoo!, and Hotmail (when using their webmail interfaces) delete the messages directly.

comment:5 Changed 2 years ago by dennylin93

  • Milestone changed from 0.4-stable to later
  • Resolution wontfix deleted
  • Status changed from closed to reopened

Hi, can someone reconsider this ticket (perhaps make it configurable)? As the email providers stated above (Gmail, Yahoo!, Hotmail) delete messages from Junk folders directly when using the webmail interface.

Last edited 2 years ago by dennylin93 (previous) (diff)

comment:6 Changed 2 years ago by alec

Rewrite your patch to make this optional.

Changed 2 years ago by dennylin93

comment:7 Changed 2 years ago by dennylin93

This new patch should work, but it may need some additional testing and clean ups.

comment:8 Changed 15 months ago by alec

  • Milestone changed from later to 0.8-rc
  • Owner set to alec
  • Status changed from reopened to new

Looks good. Would be perfect with user preference.

comment:9 Changed 15 months ago by dennylin93

I have a new patch, but I cannot upload it.

This is the error I'm seeing:

Submission rejected as potential spam (BlogSpam says content is spam (No reverse DNS entry for 147.8.249.54), Content contained these blacklisted patterns: '')

I've uploaded it the patch here: http://www.cnmc.tw/~dennylin93/tmp/delete-junk-user-configurable.patch

Changed 15 months ago by alec

Updated patch

comment:10 Changed 14 months ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Implemented in [1b30a710].

Note: See TracTickets for help on using tickets.