Paths

Table of Contentst

Differential D10022

Allow audit email to generate from multiple transactions
ClosedPublic
Actions

Authored by epriestley on Jul 22 2014, 9:30 PM.

Tags

None

Referenced Files

	F14011376: D10022.id24157.diff
	Thu, Oct 31, 11:34 PM

	F14006173: D10022.id24082.diff
	Mon, Oct 28, 2:29 AM

	F14003088: D10022.id.diff
	Sat, Oct 26, 2:02 AM

	F14001393: D10022.id24157.diff
	Fri, Oct 25, 7:21 AM

	F13991597: D10022.id24082.diff
	Tue, Oct 22, 11:44 AM

	Unknown Object (File)
	Sep 30 2024, 4:42 PM

	Unknown Object (File)
	Sep 21 2024, 5:19 AM

	Unknown Object (File)
	Sep 20 2024, 12:19 PM

Subscribers

Details

Reviewers

joshuaspence
btrahan

Maniphest Tasks

T4896: Move Audit to ApplicationTransactions

Commits

Restricted Diffusion Commit
rP970058927987: Allow audit email to generate from multiple transactions

Summary

Ref T4896. Begins laying groundwork to split comments apart so they behave like transactions, ultimately enabling the migration.

Test Plan

Made several different types of comments, verified resulting email looks OK.

Diff Detail

Repository

Branch

audit6

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 1845
Build 1846: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley updated this revision to Diff 24082.Jul 22 2014, 9:30 PM

epriestley retitled this revision from to Allow audit email to generate from multiple transactions.

epriestley updated this object.

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

epriestley added reviewers: joshuaspence, btrahan.

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

Herald added a subscriber: epriestley. · View Herald TranscriptJul 22 2014, 9:30 PM

Harbormaster completed remote builds in B1845: Diff 24082.Jul 22 2014, 9:31 PM

joshuaspence added a parent revision: D10020: Migrate Audit comment text into new storage.Jul 22 2014, 11:08 PM

joshuaspence accepted this revision.Jul 23 2014, 1:48 PM

joshuaspence edited edge metadata.

joshuaspence added inline comments.

src/applications/audit/editor/PhabricatorAuditCommentEditor.php
539	Is that any better/different to just `if ($content) {`?

This revision is now accepted and ready to land.Jul 23 2014, 1:48 PM

epriestley added inline comments.Jul 23 2014, 2:54 PM

src/applications/audit/editor/PhabricatorAuditCommentEditor.php
539	For posterity, D10020 has the reasoning -- the behavior differs on the string "0". (There's a user on GitHub with this username (https://github.com/0) and he filed an issue a while ago that his username didn't work in Phabricator. This was fixed circa rP0e672e2.)

Double-checking that lint is good.

Harbormaster completed remote builds in B1888: Diff 24148.Jul 25 2014, 12:11 AM

Closed by commit rP970058927987 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

audit/

editor/

PhabricatorAuditCommentEditor.php

39 lines

Diff 24082

src/applications/audit/editor/PhabricatorAuditCommentEditor.php

Loading...