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)
Thu, Dec 12, 4:16 PM
Unknown Object (File)
Thu, Dec 12, 5:56 AM
Unknown Object (File)
Tue, Dec 10, 5:04 PM
Unknown Object (File)
Sun, Dec 8, 2:16 AM
Unknown Object (File)
Sat, Dec 7, 2:12 PM
Unknown Object (File)
Fri, Dec 6, 6:14 AM
Unknown Object (File)
Wed, Dec 4, 8:29 AM
Unknown Object (File)
Tue, Dec 3, 8:22 PM
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.