Page MenuHomePhabricator

Allow Phame Blogs to be archived instead of deleted
ClosedPublic

Authored by chad on Nov 12 2015, 12:43 AM.
Tags
None
Referenced Files
F14060087: D14465.diff
Sun, Nov 17, 11:09 PM
F14054674: D14465.id35149.diff
Sat, Nov 16, 3:51 AM
F14054673: D14465.id34985.diff
Sat, Nov 16, 3:51 AM
F14046507: D14465.diff
Wed, Nov 13, 9:35 PM
F14039947: D14465.id34985.diff
Mon, Nov 11, 6:51 AM
F14035995: D14465.diff
Sun, Nov 10, 8:10 AM
F14031464: D14465.diff
Sat, Nov 9, 10:38 AM
F14029940: D14465.id35150.diff
Fri, Nov 8, 10:35 PM

Details

Summary

Removes "delete" and uses "archive/activate" instead for Phame Blogs. Ref T9756

Test Plan

Archive a blog, see in search, activate blog, see in other search.

Diff Detail

Repository
rP Phabricator
Branch
archive-blog
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 8963
Build 10522: Run Core Tests
Build 10521: arc lint + arc unit

Event Timeline

chad retitled this revision from to Allow Phame Blogs to be archived instead of deleted.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

Consider withStatuses(array $statuses) instead of withStatus($status). The former is a bit more flexible if we introduce other types of statuses later.

This revision is now accepted and ready to land.Nov 15 2015, 4:48 PM
chad edited edge metadata.
  • Use withStatuses
This revision was automatically updated to reflect the committed changes.