Page MenuHomePhabricator
Differential D8378

Clean up some Differential behaviors
ClosedPublic
Actions

Authored by epriestley on Mar 1 2014, 1:15 AM.
Tags
None
Referenced Files
F13173388: D8378.diff
Tue, May 7, 7:28 PM
Unknown Object (File)
Fri, May 3, 5:08 AM
Unknown Object (File)
Mon, Apr 29, 3:55 PM
Unknown Object (File)
Sun, Apr 28, 5:32 AM
Unknown Object (File)
Sun, Apr 28, 5:32 AM
Unknown Object (File)
Sun, Apr 28, 5:32 AM
Unknown Object (File)
Sat, Apr 27, 2:03 AM
Unknown Object (File)
Wed, Apr 24, 10:29 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
Commits
Restricted Diffusion Commit
rP83206242c9dd: Clean up some Differential behaviors
Summary

Ref T2222.

  • Restore mail tags for ApplicationTransactions mail.
  • Restore subject line verbs.
  • Denormalize line count and repository PHID.
  • Fix an issue with the mailgun adapter where headers weren't attached properly.
Test Plan

Sent some mail, verified it had correct subjects and tags.

Diff Detail

Repository
rP Phabricator
Branch
dxmail1
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

btrahan accepted this revision.Mar 3 2014, 4:31 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 3 2014, 4:31 PM
epriestley closed this revision.Mar 3 2014, 4:36 PM

Closed by commit rP83206242c9dd (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
editor/
4 lines
storage/
63 lines
metamta/
adapter/
5 lines

Diff 19913

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/storage/DifferentialTransaction.php

Loading...
View Options

src/applications/metamta/adapter/PhabricatorMailImplementationMailgunAdapter.php

Loading...
Phabricator · Built by Phacility