Review UX for Deletion of Record (#447)

SteveChapmanBCDX commented 3 years ago

When a record is deleted, the user is notified (by toast). However, the user remains in the record view and the deleted record is displayed.

Recommend that on deletion:

the user is notified that the record has been deleted (or not); and
if the record has been deleted, the user is redirected to the datasets list view.

SteveChapmanBCDX commented 3 years ago

@yuisotozaki - What would you recommend here? I think that most users would expect to end up back at the dataset lists view, but not sure how to handle the "record deleted" notification.

yuisotozaki commented 3 years ago

@SteveChapmanBCDX Tested the flow on toyger Version: 1.9.0-85b4524 Api Version: 2.7.5 When the user presses Delete Dataset on Dataset View:

There is no acknowledgement of the button press
There is no confirmation prompt to prevent accidental deletion
There is no explanation of what Deletion means
There is a translucent success prompt at the top right of the screen
The user is left on the deleted record's Dataset view

Based on this observation, here are my recommendations:

Redirect the user to a deletion confirmation screen and explain what deletion means; deletion means there is no way to recover this record and its resources; consider archiving if possible; only consider deletion of erroneous record creation; etc. (Input from data architects needed here)
Ask the user if they want continue with deletion with buttons "Yes" and "No"
Alternatively, we can have a textfield for the admin to type DELETE to confirm deletion to doubly ensure that there is no instinctive click on 'Yes'
The user is returned to the dataset view if No is chosen
If the user clicks on 'Yes', display an indicator that the system is doing something
When the deletion is completed, the screens shows, "Deletion completed." with a link back to the home page (by offering a link to the home page here, we cover the cases where the user simply pasted the URL in their browser to get to the dataset view, thus not having a go-back URL in the session)

annikaLiving commented 3 years ago

@yuisotozaki will add that being able to Delete a Record should only be allowed when a record is in Draft. This is currently how it is in Cat Classic. We want to encourage people once published that they Archive a record, not delete it.

ghost commented 2 years ago

https://dpdd.atlassian.net/browse/DDS-447

bcgov / ckan-ui

Review UX for Deletion of Record (#447) #320