Bug 743025 - Consistently ask users for confirmation of destructive actions
Summary: Consistently ask users for confirmation of destructive actions
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: web UI
Version: 0.7
Hardware: Unspecified
OS: Unspecified
unspecified
low vote
Target Milestone: ---
Assignee: Raymond Mancy
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-03 16:01 UTC by Derrich Hafemann
Modified: 2019-05-22 13:42 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-07-20 00:39:12 UTC


Attachments (Terms of Use)

Description Derrich Hafemann 2011-10-03 16:01:23 UTC
Description of problem:

The WebUI doesn't always ask the user to confirm when performing a destructive operation.

How reproducible: 100%


Steps to Reproduce:
1. Create group in Beaker. I called mine 'dummy'. I added a single (broken) system to it, and two users (dhafeman, gbrirner)
2. After creating group, click 'Delete'.
3. The group is deleted, with no confirmation dialogue. Of course, admin users should exercise caution, but consistency in confirmation dialogues would be good.

Comment 3 Raymond Mancy 2012-07-09 07:27:03 UTC
http://gerrit.beaker-project.org/#/c/1188/

Comment 6 Dan Callaghan 2012-07-17 05:11:34 UTC
Confirmation dialog is not displayed when deleting a group, either from the normal groups page /groups/ or the admin groups page /groups/admin. (Why do we have two separate pages?...)

Comment 7 Raymond Mancy 2012-07-17 11:38:32 UTC
This is my fault. It's very sloppy and should be fixed.

I think it was some kind of attempt to seperate administration features from regular user features (even though in it's current form they both have essentially the same features...). I'm not sure if this goal of seperation is futile (or not a good one), or whether it's worth pursuing. 

Anyway, for the time being while we have a groups/admin and groups/, 'Remove' should not be on the non admin one. I've moved it to only be on the admin one.

http://gerrit.beaker-project.org/#/c/1219/

Comment 8 Dan Callaghan 2012-07-20 00:39:12 UTC
Beaker 0.9.1 has been released.


Note You need to log in before you can comment on or make changes to this bug.