Page MenuHomePhabricator
Differential D15912

Make packages mailable and subscribable
ClosedPublic
Actions

Authored by epriestley on May 13 2016, 5:41 PM.
Tags
None
Referenced Files
F13273697: D15912.diff
Fri, May 31, 2:27 AM
F13235985: D15912.diff
Tue, May 21, 7:26 AM
F13234368: D15912.diff
Tue, May 21, 3:20 AM
F13229042: D15912.id38316.diff
Mon, May 20, 1:56 PM
F13212082: D15912.diff
Fri, May 17, 6:22 AM
F13195687: D15912.diff
Sun, May 12, 10:33 PM
F13179669: D15912.diff
Wed, May 8, 9:21 PM
F13176170: D15912.id.diff
Wed, May 8, 11:10 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T7834: Package owners doesn't get mail after a commit triggers an audit
T10939: Support for OWNERS files
Commits
rP547abfe87391: Make packages mailable and subscribable
Summary

Ref T10939. Fixes T7834.

  • Make packages into mailable objects, like projects and users.
  • Packages resolve recipients by resolving project and user owners into recipients.
Test Plan
  • Added a comment to a revision with a package subscriber.
  • Used bin/mail show-outbound to see that owners got mail.

Diff Detail

Repository
rP Phabricator
Branch
owners5
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/owners/storage/PhabricatorOwnersPackage.php:297XHP16TODO Comment
Unit
Tests Passed
Build Status
Buildable 12188
Build 15386: Run Core Tests
Build 15385: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38316.May 13 2016, 5:41 PM
epriestley retitled this revision from to Make packages mailable and subscribable.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added tasks: T10939: Support for OWNERS files, T7834: Package owners doesn't get mail after a commit triggers an audit.
chad accepted this revision.May 13 2016, 7:28 PM
chad edited edge metadata.
This revision is now accepted and ready to land.May 13 2016, 7:28 PM
Closed by commit rP547abfe87391: Make packages mailable and subscribable (authored by epriestley, committed by epriestley). · Explain WhyMay 14 2016, 12:19 AM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T10939: Support for OWNERS files.May 14 2016, 1:02 AM
epriestley mentioned this in T5000: Using Differential with plain Git, without requiring Arc.May 20 2016, 11:04 PM

Revision Contents
Changeset List

Diff 38316

View Options

src/applications/metamta/query/PhabricatorMetaMTAMemberQuery.php

Loading...
View Options

src/applications/metamta/replyhandler/PhabricatorMailTarget.php

Loading...
View Options

src/applications/metamta/typeahead/PhabricatorMetaMTAMailableDatasource.php

Loading...
View Options

src/applications/owners/controller/PhabricatorOwnersPathsController.php

Loading...
View Options

src/applications/owners/editor/PhabricatorOwnersPackageEditEngine.php

Loading...
View Options

src/applications/owners/editor/PhabricatorOwnersPackageTransactionEditor.php

Loading...
View Options

src/applications/owners/phid/PhabricatorOwnersPackagePHIDType.php

Loading...
View Options

src/applications/owners/query/PhabricatorOwnersPackageSearchEngine.php

Loading...
View Options

src/applications/owners/storage/PhabricatorOwnersPackage.php

Loading...
View Options

src/applications/owners/typeahead/PhabricatorOwnersPackageDatasource.php

Loading...
Phabricator · Built by Phacility