Apply Owners reviewers using ModularTransactions
ClosedPublic
Actions

Authored by epriestley on Mar 20 2017, 4:16 PM.

Tags

None

Referenced Files

	F18771159: D17512.diff
	Wed, Oct 8, 6:05 PM

	F18733510: D17512.id.diff
	Tue, Sep 30, 10:36 PM

	F18679069: D17512.id.diff
	Fri, Sep 26, 12:30 AM

	F18657654: D17512.diff
	Tue, Sep 23, 2:47 AM

	F18626145: D17512.id42137.diff
	Tue, Sep 16, 12:25 AM

	F18624459: D17512.id.diff
	Mon, Sep 15, 8:06 PM

	F18616517: D17512.diff
	Sun, Sep 14, 5:52 PM

	F18595217: D17512.diff
	Fri, Sep 12, 8:55 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T10967: Move Differential reviewers back to a dedicated database storage table

Commits

rPa9cbbf3e5e15: Apply Owners reviewers using ModularTransactions

Summary

Ref T10967. See that task for some discussion. This lets us do double writes on this pathway.

Test Plan

Set an Owners package to auto-review. Created revisions which triggered it: one with no reviewers (autoreview added); one with the package as a blocking reviewer explicitly (no automatic stuff happened, as expected).

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 20 2017, 4:16 PM

Harbormaster completed remote builds in B16036: Diff 42121.Mar 20 2017, 4:17 PM

chad accepted this revision.Mar 20 2017, 4:39 PM

This revision is now accepted and ready to land.Mar 20 2017, 4:39 PM

Closed by commit rPa9cbbf3e5e15: Apply Owners reviewers using ModularTransactions (authored by epriestley, committed by epriestley). · Explain WhyMar 20 2017, 11:43 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

editor/

DifferentialTransactionEditor.php

17 lines

Diff 42137

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Apply Owners reviewers using ModularTransactionsClosedPublicActions