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
F13842765: D8734.diff
Thu, Sep 26, 11:13 PM
Unknown Object (File)
Tue, Sep 3, 9:20 PM
Unknown Object (File)
Aug 21 2024, 11:35 PM
Unknown Object (File)
Aug 21 2024, 11:35 PM
Unknown Object (File)
Aug 19 2024, 3:52 PM
Unknown Object (File)
Aug 15 2024, 9:19 AM
Unknown Object (File)
Aug 14 2024, 9:46 PM
Unknown Object (File)
Aug 10 2024, 6:55 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
Branch
master
Lint
Lint Passed
Unit
Tests Passed

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).