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
F14060690: D15220.diff
Mon, Nov 18, 3:00 AM
F14048004: D15220.diff
Thu, Nov 14, 6:21 AM
F14033937: D15220.diff
Sat, Nov 9, 8:52 PM
F14018682: D15220.diff
Tue, Nov 5, 6:22 PM
F13984140: D15220.diff
Sun, Oct 20, 11:14 AM
F13968680: D15220.id36740.diff
Oct 16 2024, 10:42 PM
Unknown Object (File)
Oct 7 2024, 11:16 PM
Unknown Object (File)
Sep 23 2024, 9:31 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.