Differential D13750

Add Priority to Maniphest Hovercard
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F18820542: D13750.diff
	Wed, Oct 22, 3:49 PM

	F18779779: D13750.diff
	Sat, Oct 11, 3:23 PM

	F18740490: D13750.id33216.diff
	Thu, Oct 2, 3:36 AM

	F18740486: D13750.id.diff
	Thu, Oct 2, 3:26 AM

	F18740463: D13750.diff
	Thu, Oct 2, 3:20 AM

	F18484716: D13750.id.diff
	Sep 3 2025, 5:04 PM

	F18468500: D13750.diff
	Sep 2 2025, 3:24 PM

	F18091965: D13750.id33216.diff
	Aug 7 2025, 3:16 AM

Subscribers

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

Branch

hovercard-priority

Lint

Lint Passed

Unit

Test Failures

Build Status

Buildable 7435
Build 7910: [Placeholder Plan] Wait for 30 Seconds
Build 7909: arc lint + arc unit

Unit TestsFailed
View All

	Time	Test
	58 ms	ManiphestTaskTestCase::Unknown Unit Message ("")
	9 ms	ManiphestTaskTestCase::Unknown Unit Message ("")
	1,610 ms	PhabricatorCelerityTestCase::Unknown Unit Message ("")
	9 ms	PhabricatorConduitTestCase::Unknown Unit Message ("")
	0 ms	PhabricatorInfrastructureTestCase::Unknown Unit Message ("")
		View Full Test Results (2 Failed · 7 Passed)

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 33211

src/applications/maniphest/event/ManiphestHovercardEventListener.php

Loading...