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
F14067624: D15912.diff
Tue, Nov 19, 3:57 PM
F14053145: D15912.diff
Fri, Nov 15, 12:54 PM
F14040354: D15912.diff
Mon, Nov 11, 9:25 AM
F14022842: D15912.diff
Wed, Nov 6, 8:06 PM
F14014947: D15912.diff
Sun, Nov 3, 10:49 AM
F13997742: D15912.diff
Thu, Oct 24, 5:57 AM
F13978331: D15912.id38328.diff
Oct 18 2024, 10:43 PM
F13976485: D15912.diff
Oct 18 2024, 2:29 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
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