Bug 743025

Summary:	Consistently ask users for confirmation of destructive actions
Product:	[Retired] Beaker	Reporter:	Derrich Hafemann <dhafeman>
Component:	web UI	Assignee:	Raymond Mancy <rmancy>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	0.7	CC:	bpeck, dcallagh, ebaak, mcsontos, mschick, rmancy, stl
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2012-07-20 00:39:12 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

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.