Page MenuHomePhabricator

Support enabling a formal points field in Maniphest
ClosedPublic

Authored by epriestley on Feb 9 2016, 12:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:17 PM
Unknown Object (File)
Wed, Apr 24, 12:01 AM
Unknown Object (File)
Fri, Apr 19, 2:24 AM
Unknown Object (File)
Fri, Apr 19, 2:24 AM
Unknown Object (File)
Fri, Apr 19, 2:24 AM
Unknown Object (File)
Fri, Apr 19, 2:24 AM
Unknown Object (File)
Sun, Apr 14, 10:49 PM
Unknown Object (File)
Thu, Apr 11, 3:44 AM
Subscribers
None

Details

Summary

Ref T4427.

  • New config option for labels, enabling, etc., but no UI/niceness yet.
  • When enabled, add a field.
  • Allow nonnegative values, including fractional values.
  • EditEngine is nice and Conduit / actions basically just work with a tiny bit of extra support code.
Test Plan
  • Edited points via "Edit".
  • Edited points via Conduit.
  • Edited points via stacked actions.
  • Tried to set "zebra" points.
  • Tried to set -1 points.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Support enabling a formal points field in Maniphest.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

This doesn't do anything on projects/workboards yet, that'll be coming soon.

chad edited edge metadata.
This revision is now accepted and ready to land.Feb 9 2016, 2:10 AM
src/aphront/AphrontRequest.php
200 ↗(On Diff #36740)

readImage.jpg (323×575 px, 25 KB)

src/aphront/AphrontRequest.php
200 ↗(On Diff #36740)

oh well you see

it's complicated

shhhh

epriestley edited edge metadata.
  • No random newline in unrelated file.
This revision was automatically updated to reflect the committed changes.