Page MenuHomePhabricator

Consolidate outbound mail status in a new class

Authored by epriestley on Aug 13 2015, 6:13 PM.



Ref T5791. This collects outbound mail status in one place and makes the list view a little spiffier.

Test Plan

Looked at list and detail views. Grepped for changed classes/constants.

Diff Detail

rP Phabricator
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley updated this revision to Diff 33520.Aug 13 2015, 6:13 PM
epriestley retitled this revision from to Consolidate outbound mail status in a new class.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad accepted this revision.Aug 13 2015, 6:48 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Aug 13 2015, 6:48 PM
chad added inline comments.Aug 13 2015, 8:34 PM






chad added a comment.Aug 13 2015, 8:34 PM

I played with this a little bit today, and it was awesome. Not sure the tabs are needed, do you think more this page's information will continue to grow?

There's at least a bit more stuff coming, yeah (at a minimum: exposing mail tags more clearly, adding Herald routing information).

I also imagine this may start looking and working a little more like your Gmail inbox eventually, where some users might read some of their mail directly from the web UI, so I'm trying to bend it in that direction a bit. I think the primary use case for this UI will be "understand what happened to mail", at least for a while, but I'd like to see if "read mail" can also eventually be a major use case. Stuff like "mail headers" is a little bit helpful in understanding what happened to mail, but totally useless when reading mail, which is why I moved it to a separate tab and tried to make the main view look about as much like an inbox message as it can.

epriestley edited edge metadata.
  • Use different icons for different statuses.
This revision was automatically updated to reflect the committed changes.