Apply Owners reviewers using ModularTransactions
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13090198: D17512.diff
	Thu, Apr 25, 2:15 AM

	Unknown Object (File)
	Fri, Apr 19, 10:31 PM

	Unknown Object (File)
	Mon, Apr 8, 8:44 AM

	Unknown Object (File)
	Feb 14 2024, 2:06 PM

	Unknown Object (File)
	Jan 26 2024, 8:19 AM

	Unknown Object (File)
	Jan 21 2024, 5:44 AM

	Unknown Object (File)
	Dec 27 2023, 10:15 AM

	Unknown Object (File)
	Dec 25 2023, 10:24 AM

View All 72 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