Page MenuHomePhabricator

Add a priorityColor property to the maniphest conduit endpoint
ClosedPublic

Authored by bwinterton on Apr 9 2014, 6:02 PM.
Tags
None
Referenced Files
F14052366: D8734.diff
Fri, Nov 15, 8:37 AM
F14051982: D8734.id20717.diff
Fri, Nov 15, 5:13 AM
F14041095: D8734.diff
Mon, Nov 11, 3:55 PM
F14024414: D8734.diff
Thu, Nov 7, 9:17 AM
F13989148: D8734.id.diff
Mon, Oct 21, 6:25 PM
F13987517: D8734.id20717.diff
Mon, Oct 21, 9:52 AM
Unknown Object (File)
Oct 13 2024, 1:16 AM
Unknown Object (File)
Oct 10 2024, 7:15 PM
Subscribers

Details

Summary

I added a getTaskPriorityColor function to the ManiphestTaskPriority class which returns the color set in the maniphest config for the given priority.

This is in preparation for a change to arcanist which will allow it to display the priority color (if it is a supported color) upon running arc tasks.

Fixed some linting issues

Test Plan

Invoke the maniphest.info method from conduit and ensure that:

  • The priorityColor property is given in the json
  • the priorityColor property is set correctly

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

bwinterton retitled this revision from to Add a priorityColor property to the maniphest conduit endpoint.
bwinterton updated this object.
bwinterton edited the test plan for this revision. (Show Details)
bwinterton added a reviewer: epriestley.
This revision is now accepted and ready to land.Apr 9 2014, 6:31 PM
epriestley updated this revision to Diff 20717.

Closed by commit rPb660960771a4 (authored by @bwinterton, committed by @epriestley).