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
F15541783: D8378.diff
Fri, Apr 25, 1:31 PM
F15537883: D8378.diff
Thu, Apr 24, 4:18 PM
F15530602: D8378.diff
Wed, Apr 23, 6:18 AM
F15527031: D8378.diff
Tue, Apr 22, 12:13 AM
F15517644: D8378.id19926.diff
Sat, Apr 19, 7:23 AM
F15511632: D8378.id.diff
Thu, Apr 17, 3:47 AM
F15509750: D8378.diff
Wed, Apr 16, 3:51 PM
F15441010: D8378.id.diff
Mar 26 2025, 4:04 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