Add Badges to Hovercards
ClosedPublic
Actions

Authored by chad on Jul 24 2015, 4:57 PM.

Details

Reviewers

epriestley
btrahan

Maniphest Tasks

T8942: Add badges to hovercards

Commits

Restricted Diffusion Commit
rP0ba65699596d: Add Badges to Hovercards

Summary

Fixes T8942, Makes Badges if installed show up on your hovercard.

Test Plan

Give a badge, mention myself, see badge on hovercard.

Diff Detail

Repository

rP Phabricator

Branch

badges-hovercard

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7380
Build 7800: [Placeholder Plan] Wait for 30 Seconds
Build 7799: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33115.Jul 24 2015, 4:57 PM

chad retitled this revision from to Add Badges to Hovercardsy.

chad updated this object.

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

chad added reviewers: epriestley, btrahan.

chad added a task: T8942: Add badges to hovercards.

Herald added subscribers: Korvin, epriestley. · View Herald TranscriptJul 24 2015, 4:57 PM

chad retitled this revision from Add Badges to Hovercardsy to Add Badges to Hovercards.Jul 24 2015, 4:57 PM

epriestley accepted this revision.Jul 24 2015, 5:53 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jul 24 2015, 5:53 PM

Closed by commit rP0ba65699596d: Add Badges to Hovercards (authored by chad, committed by chad). · Explain WhyJul 24 2015, 5:58 PM

This revision was automatically updated to reflect the committed changes.

joshuaspence awarded a token.Jul 25 2015, 1:25 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

people/

event/

PhabricatorPeopleHovercardEventListener.php

32 lines

Diff 33115

View Options

src/applications/people/event/PhabricatorPeopleHovercardEventListener.php

Add Badges to HovercardsClosedPublicActions