pazz / alot

Terminal-based Mail User Agent
GNU General Public License v3.0
681 stars 162 forks source link

Add option to remove obsolete drafts #1627

Open milhnl opened 10 months ago

milhnl commented 10 months ago

Hi all!

This is an updated version of https://github.com/pazz/alot/pull/1313 (which aimed to fix https://github.com/pazz/alot/issues/1044). I rebased the commit, and deduplicated the code in SaveCommand and SendCommand. I think that was the most obvious feedback then, but I can imagine there's more.

milhnl commented 10 months ago

By the way, I just remembered I saw a pull request today that I think was merged which made the tags to indicate a draft configurable by the user. That is not yet incorporated in this commit.

guludo commented 4 months ago

I took a look at adding support for the draft_tags stuff here and it seems there are other places in the current code base that needs the same type of fix already (see the output of git grep -W -e "'draft'" -e '"draft"' upstream/master)...