Page MenuHomePhabricator

Only add the Mail "STAMPS" body section if there are stamps
ClosedPublic

Authored by epriestley on Feb 8 2018, 5:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 6, 2:12 AM
Unknown Object (File)
Fri, Sep 6, 2:12 AM
Unknown Object (File)
Fri, Sep 6, 2:12 AM
Unknown Object (File)
Mon, Sep 2, 6:47 AM
Unknown Object (File)
Sun, Sep 1, 8:34 AM
Unknown Object (File)
Sun, Sep 1, 8:33 AM
Unknown Object (File)
Sun, Sep 1, 8:33 AM
Unknown Object (File)
Wed, Aug 28, 1:59 PM
Subscribers
None

Details

Summary

Ref T13053. Some mail (like push notification mail) doesn't currently generate any stamps. Drop this section if there aren't any stamps on the mail.

Test Plan

Will check push mail in production.

Diff Detail

Repository
rP Phabricator
Branch
mail3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19421
Build 26265: Run Core Tests
Build 26264: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2018, 5:09 PM
This revision was automatically updated to reflect the committed changes.