Page MenuHomePhabricator
Create Task
Maniphest T5658

Change "Pending Differential Revisions" into "Recent Pending Differential Revisions"
Closed, ResolvedPublic
Actions

Description

D9306 adds a config option to allow non-authors to abandon revisions. Possibly admins should always be able to perform this action, as a way to clean up data on an install.

In particular, this install has a lot of stale revisions that should be abandoned. These don't really do any harm, but they do mess up the "open revisions affecting these files" stuff.

Revisions and Commits

rP Phabricator
D12158rPe5445de1635f Show only recent open revisions affecting the same files

Related Objects

Event Timeline

joshuaspence created this task.Jul 19 2014, 11:38 PM
joshuaspence claimed this task.
joshuaspence raised the priority of this task from to Needs Triage.
joshuaspence updated the task description. (Show Details)
joshuaspence added a project: Differential.
joshuaspence added subscribers: joshuaspence, epriestley.
joshuaspence added a comment.Jul 21 2014, 8:49 AM

Possibly this is better suited for a policy rather than a config option.

epriestley added a comment.Jul 21 2014, 1:19 PM

If the major issue is "Open Revisions Affecting These Files", maybe a better approach would be to change that to "Open Revisions Affecting These Files, Updated in the Last 30 Days"?

joshuaspence added a comment.Sep 13 2014, 4:35 AM

I would be happy with that

cburroughs added a subscriber: cburroughs.Sep 14 2014, 1:50 AM

I like the idea of a policy option to let me unleash excessively attentive people on our install to keep it clean without giving every user a way to mess things up.

joshuaspence removed joshuaspence as the assignee of this task.Oct 12 2014, 8:10 PM
epriestley renamed this task from Always allow admins to abandon a revision to Change "Pending Differential Revisions" into "Recent Pending Differential Revisions".Nov 6 2014, 11:05 PM
epriestley mentioned this in T6486: Pending Differential Revisions isn't filtered by branch.
chad mentioned this in T6853: "Open Revisions Affecting These Files" should weight severity of changes.Jan 2 2015, 10:43 PM
epriestley moved this task from Backlog to v2 (UI/Mobile) on the Differential board.Mar 4 2015, 5:12 PM
epriestley added a revision: D12158: Show only recent open revisions affecting the same files.Mar 25 2015, 1:53 PM
epriestley claimed this task.Mar 25 2015, 1:53 PM
epriestley triaged this task as Normal priority.
epriestley closed this task as Resolved by committing rPe5445de1635f: Show only recent open revisions affecting the same files.Mar 25 2015, 5:21 PM
epriestley added a commit: rPe5445de1635f: Show only recent open revisions affecting the same files.
Phabricator · Built by Phacility