New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 618555 link

Starred by 1 user

Issue metadata

Status: Archived
Owner: ----
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

Warn CL owner if they are deleting a patch for which there are uncommitted commetns

Project Member Reported by pkotw...@chromium.org, Jun 9 2016

Issue description

This happened once to me and was super annoying.
1) Start doing a code review
2) CL owner deletes patch set because they have uploaded a new patch set

Expected:
CL owner gets alert telling them that they are about to delete patch set for which there are pending comments
Actual:
There is special warning. when CL owner clicks link to delete patch set. All of my pending comments are gone!

I delete old patch sets whenever I upload a new patch set and no one has commented on my old patch set because I think that it makes the change list cleaner. This feature would make my life much better


 
Labels: Infra-DX
Status: Available (was: Untriaged)
Rietveld is currently in maintenance mode while we transition to Gerrit. A broader announcement will be made shortly. This bug will be kept open to ensure that the behavior doesn’t manifest itself in the new tool.
Status: Archived (was: Available)
Pri-3 with no updates in 180+ days that are available and have no owner == Archived. Feel free to update status, add context as to why and move this along.

Sign in to add a comment