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
Unknown Object (File)
Tue, Jun 17, 10:14 PM
Unknown Object (File)
Thu, Jun 5, 2:39 PM
Unknown Object (File)
Thu, Jun 5, 12:47 PM
Unknown Object (File)
Wed, Jun 4, 11:23 AM
Unknown Object (File)
Tue, Jun 3, 7:45 PM
Unknown Object (File)
Sun, Jun 1, 1:35 PM
Unknown Object (File)
Sun, Jun 1, 1:35 PM
Unknown Object (File)
Sun, Jun 1, 1:35 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 38328

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