Page MenuHomePhabricator
Differential D20635

Move workboard "Move Tasks to Column..." workflow to a separate controller
ClosedPublic
Actions

Authored by epriestley on Jul 2 2019, 2:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 4:24 AM
Unknown Object (File)
Thu, Apr 11, 4:23 AM
Unknown Object (File)
Thu, Apr 11, 4:23 AM
Unknown Object (File)
Wed, Apr 10, 6:27 PM
Unknown Object (File)
Tue, Apr 2, 10:37 AM
Unknown Object (File)
Mon, Apr 1, 1:21 PM
Unknown Object (File)
Sat, Mar 30, 8:20 AM
Unknown Object (File)
Sat, Mar 30, 8:20 AM
View All Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T4900: Workboards updating in real time
T13316: On Workboards, "Move Tasks to Column..." has high friction in the common use case (selecting a column on the same board)
Commits
rP24b466cd62d1: Move workboard "Move Tasks to Column..." workflow to a separate controller
Summary

Depends on D20634. Ref T4900. Ref T13316. I'm planning to do a bit of additional cleanup here in followups, but this separates the main workflow out of the common controller.

Test Plan
  • Used "Move Tasks to Column..." to move some tasks on a board.
  • Tried to move an empty column, hit an error.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

Diff 49234

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/project/application/PhabricatorProjectApplication.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectBoardViewController.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectColumnBulkMoveController.php

Loading...
Phabricator · Built by Phacility