Page MenuHomePhabricator

Maniphest / Workboards - show workboard column when adding a project to a task
ClosedPublic

Authored by btrahan on Jan 6 2015, 10:12 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 17, 7:57 PM
Unknown Object (File)
Thu, Apr 11, 6:54 AM
Unknown Object (File)
Sun, Apr 7, 2:48 PM
Unknown Object (File)
Thu, Mar 28, 10:11 PM
Unknown Object (File)
Tue, Mar 26, 5:03 PM
Unknown Object (File)
Tue, Mar 26, 3:21 PM
Unknown Object (File)
Mar 20 2024, 3:40 PM
Unknown Object (File)
Feb 7 2024, 11:30 AM
Subscribers

Details

Summary

Fixes T6471. This makes adding projects to tasks have better workflow towards boards; without it, you have to click project -> board -> do stuff on board as opposed to column -> do stuff on board.

Test Plan

added and removed projets. saw column listed parenthetcally when expected

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Maniphest / Workboards - show workboard column when adding a project to a task.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 6 2015, 10:30 PM

This is vaaaguely bad if some project ever has 50000 columns, but we can cross that bridge when we come to it.

We can limit the columns query to just active columns maybe?

We can assume that its going to be the default column if a default column exists and there's no position, but getting the default column still requires getting all columns as far as I know. Ergo, still need to load all active columns I think...

This revision was automatically updated to reflect the committed changes.

I don't think it's a problem for today.

5 years from now some project might have (5 * 50 ~= 250) sprint subprojects which generate columns or something, and maybe they never get auto-archived, but we have a few years before we're likely to run into that on realistic installs.