Page MenuHomePhabricator
Differential D15590

Adding awarder info to recipient list on badge view
ClosedPublic
Actions

Authored by lpriestley on Apr 3 2016, 2:19 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 18, 3:19 AM
Unknown Object (File)
Wed, Nov 13, 9:32 PM
Unknown Object (File)
Sat, Nov 9, 6:34 PM
Unknown Object (File)
Tue, Nov 5, 9:51 PM
Unknown Object (File)
Wed, Oct 30, 5:15 AM
Unknown Object (File)
Wed, Oct 30, 3:58 AM
Unknown Object (File)
Thu, Oct 24, 8:01 AM
Unknown Object (File)
Oct 18 2024, 4:00 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8940: Annotate badges with awarder, date
Commits
rP88d15ce79910: Adding awarder info to recipient list on badge view
Summary

Closes T8940, recipient list in badge view should show awarder and date info. Took a first stab at how we want to make the date look, but not sure. Looks odd as it is.

Test Plan

Open badge that has awards. Each recipient in list should have a subheader such as "Awarded by ... on ..."

Diff Detail

Repository
rP Phabricator
Branch
badgeviewawarderinfo
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 11448
Build 14273: Run Core Tests
Build 14272: arc lint + arc unit

Event Timeline

lpriestley updated this revision to Diff 37580.Apr 3 2016, 2:19 AM
lpriestley retitled this revision from to Adding awarder info to recipient list on badge view.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8940: Annotate badges with awarder, date.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 3 2016, 2:19 AM
Herald added a subscriber: Korvin. · View Herald Transcript
epriestley accepted this revision.Apr 3 2016, 2:35 AM
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 3 2016, 2:35 AM
Closed by commit rP88d15ce79910: Adding awarder info to recipient list on badge view (authored by lpriestley, committed by lpriestley). · Explain WhyApr 3 2016, 3:20 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
badges/
view/
10 lines

Diff 37580

View Options

src/applications/badges/view/PhabricatorBadgesRecipientsListView.php

Loading...
Phabricator · Built by Phacility