Page MenuHomePhabricator
Differential D10256

Add an X-Phabricator-Projects header to outbound mail
ClosedPublic
Actions

Authored by epriestley on Aug 13 2014, 6:38 PM.
Tags
None
Referenced Files
F15491932: D10256.diff
Sat, Apr 12, 9:05 AM
F15487348: D10256.id24690.diff
Thu, Apr 10, 1:20 PM
F15481347: D10256.id24697.diff
Tue, Apr 8, 8:13 PM
F15476900: D10256.id.diff
Mon, Apr 7, 10:02 AM
F15474388: D10256.diff
Sun, Apr 6, 8:20 AM
F15391264: D10256.id24697.diff
Mar 15 2025, 9:06 AM
F15387841: D10256.id24697.diff
Mar 15 2025, 3:00 AM
F15385118: D10256.id24690.diff
Mar 14 2025, 9:45 PM
View All Files
Subscribers
• chasemp
epriestley

Details

Reviewers
btrahan
Maniphest Tasks
T4973: Add project names to a header in outbound Maniphest emails
Commits
Restricted Diffusion Commit
rPdf28d751f587: Add an X-Phabricator-Projects header to outbound mail
Summary

Fixes T4973. For PhabricatorProjectInterface objects, add a header to let clients do mail filtering.

Test Plan

Saw X-Phabricator-Projects: <#goat_farm> in outbound mail.

Diff Detail

Repository
rP Phabricator
Branch
projheader
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php:1924XHP16TODO Comment
Unit
No Test Coverage
Build Status
Buildable 2210
Build 2214: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24690.Aug 13 2014, 6:38 PM
epriestley retitled this revision from to Add an X-Phabricator-Projects header to outbound mail.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
epriestley added a task: T4973: Add project names to a header in outbound Maniphest emails.
Herald added a subscriber: epriestley. · View Herald TranscriptAug 13 2014, 6:38 PM
Harbormaster completed remote builds in B2210: Diff 24690.Aug 13 2014, 6:39 PM
btrahan accepted this revision.Aug 13 2014, 9:16 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Aug 13 2014, 9:16 PM
• chasemp added a subscriber: • chasemp.Aug 13 2014, 9:20 PM

you guys are awesome!

epriestley closed this revision.Aug 13 2014, 9:41 PM
epriestley updated this revision to Diff 24697.

Closed by commit rPdf28d751f587 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
src/
applications/
transactions/
editor/
48 lines

Diff 24690

View Options

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...
Phabricator · Built by Phacility