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
F18107023: D8378.id.diff
Sun, Aug 10, 11:36 PM
F17938597: D8378.id19926.diff
Thu, Jul 31, 2:26 AM
F17925078: D8378.id19926.diff
Wed, Jul 30, 10:56 AM
F17894497: D8378.id.diff
Tue, Jul 29, 5:48 AM
F17870090: D8378.diff
Mon, Jul 28, 1:06 PM
F17845375: D8378.diff
Sun, Jul 27, 3:23 AM
Unknown Object (File)
Jun 29 2025, 5:22 AM
Unknown Object (File)
Jun 28 2025, 5:09 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

Lint
Lint Skipped
Unit
Tests Skipped

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 19926

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