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
F15527798: D8734.id20717.diff
Tue, Apr 22, 6:37 AM
F15488850: D8734.id20717.diff
Fri, Apr 11, 3:28 AM
F15485068: D8734.diff
Wed, Apr 9, 9:34 PM
F15480080: D8734.id20714.diff
Tue, Apr 8, 11:26 AM
F15479910: D8734.id.diff
Tue, Apr 8, 10:09 AM
F15476351: D8734.diff
Mon, Apr 7, 5:49 AM
F15424970: D8734.id20714.diff
Mar 23 2025, 1:27 AM
F15393712: D8734.id20714.diff
Mar 15 2025, 11:03 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).