Paths

Table of Contentst

Differential D20286

Modularize workboard column transactions
ClosedPublic
Actions

Authored by epriestley on Mar 14 2019, 3:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 14, 10:03 AM

	Unknown Object (File)
	Sun, Dec 8, 4:32 PM

	Unknown Object (File)
	Fri, Dec 6, 5:55 AM

	Unknown Object (File)
	Wed, Dec 4, 4:51 AM

	Unknown Object (File)
	Tue, Dec 3, 12:46 AM

	Unknown Object (File)
	Tue, Nov 26, 7:03 PM

	Unknown Object (File)
	Nov 22 2024, 4:48 AM

	Unknown Object (File)
	Nov 18 2024, 4:29 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T5474: Support workboard column triggers which activate when a task is dropped into a column

Commits

rP0204489a5265: Modularize workboard column transactions

Summary

Depends on D20279. Ref T5474. Modernize these transactions before I add a new "TriggerTransaction" for setting triggers.

Test Plan

Created a column. Edited a column name and point limit. Hid and un-hid a column. Grepped for removed symbols.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 14 2019, 3:22 PM

Harbormaster completed remote builds in B22288: Diff 48425.Mar 14 2019, 3:24 PM

epriestley requested review of this revision.Mar 14 2019, 3:24 PM

epriestley added inline comments.Mar 14 2019, 3:25 PM

src/applications/project/controller/PhabricatorProjectColumnEditController.php
96	This fixes a bug where this workflow didn't actually enforce the requirement that columns have a name.

epriestley added a child revision: D20287: Allow triggers to be attached to and removed from workboard columns.Mar 14 2019, 5:23 PM

amckinley accepted this revision.Mar 18 2019, 6:57 PM

amckinley added inline comments.

src/applications/project/xaction/column/PhabricatorProjectColumnLimitTransaction.php
40	For consistency, "changed the point limit".
src/applications/project/xaction/column/PhabricatorProjectColumnStatusTransaction.php
23	"unhid this column"
27	"hid this column"
48	This is just going to be `"0, 1"`; maybe don't even bother to show it?

This revision is now accepted and ready to land.Mar 18 2019, 6:57 PM

epriestley added inline comments.Mar 25 2019, 6:48 PM

src/applications/project/xaction/column/PhabricatorProjectColumnStatusTransaction.php
48	I think it'll be real strings some day. Showing it is mostly for the benefit of API callers, since the internal constants may not always be obvious.

Wordsmithing.

Harbormaster completed remote builds in B22361: Diff 48489.Mar 25 2019, 6:51 PM

Closed by commit rP0204489a5265: Modularize workboard column transactions (authored by epriestley). · Explain WhyMar 25 2019, 8:14 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

applications/

project/

controller/

PhabricatorProjectColumnEditController.php

5 lines

PhabricatorProjectColumnHideController.php

4 lines

editor/

PhabricatorProjectColumnTransactionEditor.php

126 lines

storage/

PhabricatorProjectColumnTransaction.php

70 lines

xaction/

column/

PhabricatorProjectColumnLimitTransaction.php

63 lines

PhabricatorProjectColumnNameTransaction.php

66 lines

PhabricatorProjectColumnStatusTransaction.php

55 lines

PhabricatorProjectColumnTransactionType.php

4 lines

Diff 48494

src/__phutil_library_map__.php

Loading...

src/applications/project/controller/PhabricatorProjectColumnEditController.php

Loading...

src/applications/project/controller/PhabricatorProjectColumnHideController.php

Loading...

src/applications/project/editor/PhabricatorProjectColumnTransactionEditor.php

Loading...

src/applications/project/storage/PhabricatorProjectColumnTransaction.php

Loading...

src/applications/project/xaction/column/PhabricatorProjectColumnLimitTransaction.php

Loading...

src/applications/project/xaction/column/PhabricatorProjectColumnNameTransaction.php

Loading...

src/applications/project/xaction/column/PhabricatorProjectColumnStatusTransaction.php

Loading...

src/applications/project/xaction/column/PhabricatorProjectColumnTransactionType.php

Loading...