Page MenuHomePhabricator
Differential D15239

Allow "0" to be a valid workboard column point limit
ClosedPublic
Actions

Authored by epriestley on Feb 10 2016, 11:08 PM.

Details

Reviewers
chad
Maniphest Tasks
T6580: allow workboard columns to have a point limit of 0
Commits
Restricted Diffusion Commit
rP0b5abf7bb51a: Allow "0" to be a valid workboard column point limit
Summary

Fixes T6580. Now:

  • Empty field means "unlimited".
  • Zero means 0.
  • Nonzero means that number.

(Although you can now have fractional points, I didn't change columns to allow fractional limits, because too bad.)

Test Plan

Screen Shot 2016-02-10 at 3.06.11 PM.png (1×1 px, 116 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 36782.Feb 10 2016, 11:08 PM
epriestley retitled this revision from to Allow "0" to be a valid workboard column point limit.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T6580: allow workboard columns to have a point limit of 0.
chad accepted this revision.Feb 10 2016, 11:09 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Feb 10 2016, 11:09 PM
Closed by commit rP0b5abf7bb51a: Allow "0" to be a valid workboard column point limit (authored by epriestley, committed by epriestley). · Explain WhyFeb 10 2016, 11:10 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
66 lines
src/
applications/
project/
controller/
9 lines
editor/
10 lines

Diff 36783

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectColumnDetailController.php

Loading...
View Options

src/applications/project/editor/PhabricatorProjectColumnTransactionEditor.php

Loading...
Phabricator · Built by Phacility