Add Priority to Maniphest Hovercard
ClosedPublic
Actions

Authored by chad on Jul 28 2015, 10:12 PM.

Details

Reviewers

epriestley
lpriestley

Maniphest Tasks

T5524: Show task priority in task hovercards in remarkup

Commits

Restricted Diffusion Commit
rPfb13692bd3e5: Add Priority to Maniphest Hovercard

Summary

Fixes T5524, adds priority to Maniphest's hovercard. Also fixes Blocking language.

Test Plan

Hover over a task, see new stuff.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad updated this revision to Diff 33211.Jul 28 2015, 10:12 PM

chad retitled this revision from to Add Priority to Maniphest Hovercard.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added reviewers: epriestley, lpriestley.

chad added a task: T5524: Show task priority in task hovercards in remarkup.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJul 28 2015, 10:12 PM

The unit test fail is that I have custom fields that are required.

thumbsup

epriestley accepted this revision.Jul 29 2015, 12:18 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jul 29 2015, 12:18 AM

Closed by commit rPfb13692bd3e5: Add Priority to Maniphest Hovercard (authored by chad, committed by chad). · Explain WhyJul 29 2015, 2:42 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

maniphest/

event/

ManiphestHovercardEventListener.php

7 lines

Diff 33216

View Options

src/applications/maniphest/event/ManiphestHovercardEventListener.php

Add Priority to Maniphest HovercardClosedPublicActions