Page MenuHomePhabricator

Add PhabricatorBadgeArchiveController
ClosedPublic

Authored by chad on Dec 9 2015, 9:15 PM.

Details

Summary

Allows archive and activate on badges from action list. Ref T9414

Test Plan

Archive, Activate, New, Edit

Diff Detail

Repository
rP Phabricator
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

chad updated this revision to Diff 35614.Dec 9 2015, 9:15 PM
chad retitled this revision from to Add PhabricatorBadgeArchiveController.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley accepted this revision.Dec 9 2015, 9:21 PM
epriestley edited edge metadata.

Maybe don't show archived badges on profile/timeline? Or if we already don't, mention that effect in the dialog?

src/applications/badges/controller/PhabricatorBadgesArchiveController.php
56–58

he he he

This revision is now accepted and ready to land.Dec 9 2015, 9:21 PM
chad added a comment.Dec 9 2015, 9:24 PM

I'll file a task to track that.

chad updated this revision to Diff 35616.Dec 9 2015, 9:27 PM
chad edited edge metadata.
  • clean up language
This revision was automatically updated to reflect the committed changes.