Use transactions to apply "resign" and "close" Audit actions
ClosedPublic
Actions

Authored by epriestley on Aug 2 2014, 6:46 PM.

Details

Reviewers

joshuaspence
btrahan

Maniphest Tasks

T4896: Move Audit to ApplicationTransactions

Commits

Restricted Diffusion Commit
rP78e164aea6e0: Use transactions to apply "resign" and "close" Audit actions

Summary

Ref T4896. Hook these up with new stuff.

Test Plan

Closed an audit.
Resigned from an audit.

Diff Detail

Repository

rP Phabricator

Branch

audit2

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2006
Build 2007: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24351.Aug 2 2014, 6:46 PM

epriestley retitled this revision from to Use transactions to apply "resign" and "close" Audit actions.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added reviewers: btrahan, joshuaspence.

epriestley added a task: T4896: Move Audit to ApplicationTransactions.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 2 2014, 6:46 PM

Harbormaster completed remote builds in B2006: Diff 24351.Aug 2 2014, 6:47 PM

btrahan accepted this revision.Aug 2 2014, 7:13 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Aug 2 2014, 7:13 PM

Closed by commit rP78e164aea6e0 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

editor/

PhabricatorAuditCommentEditor.php

37 lines

PhabricatorAuditEditor.php

113 lines

Diff 24351

View Options

src/applications/audit/editor/PhabricatorAuditCommentEditor.php