Page MenuHomePhabricator

Make deleting a blog a little easier to recover from
ClosedPublic

Authored by chad on Nov 10 2015, 7:52 PM.
Tags
None
Referenced Files
F15484361: D14457.diff
Wed, Apr 9, 5:55 PM
F15380137: D14457.id34958.diff
Mar 14 2025, 1:36 AM
F15375483: D14457.id34958.diff
Mar 12 2025, 11:31 PM
F15361631: D14457.id.diff
Mar 11 2025, 12:37 PM
F15336850: D14457.diff
Mar 9 2025, 2:40 AM
F15285405: D14457.diff
Mar 4 2025, 1:30 PM
Unknown Object (File)
Feb 14 2025, 3:51 AM
Unknown Object (File)
Feb 8 2025, 11:46 PM
Subscribers

Details

Summary

We currently orphan posts when you delete a blog. Fixes some visibility and permission errors when that happens. Also... should allow you to archive posts.

Test Plan

Delete a blog, visit a post I made, still can see it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Make deleting a blog a little easier to recover from.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

We should disable this stuff rather than deleting it, and PostQuery should just refuse to load posts that don't have a blog. But we can fix that up later.

This revision is now accepted and ready to land.Nov 10 2015, 7:55 PM
This revision was automatically updated to reflect the committed changes.

ah true, then we don't orphan the post, it just stays on an archived blog.