otwstephanie / otwarchive3

Clone of OTWarchive, for merge testing
0 stars 0 forks source link

User Profile needs a 'delete account' option #363

Closed otwstephanie closed 10 years ago

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on September 14, 2008 04:28:21

What archive revision are you testing on? (rev. 674 #)

A user should be able to delete her account!

See the wiki for details. http://wiki.transformativeworks.org/mediawiki/Profile

Original issue: http://code.google.com/p/otwarchive/issues/detail?id=363

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 14, 2008 09:45:17

Starting to look into this!

Status: Started
Owner: sarkymoocow

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 20, 2008 16:11:24

Revision 803 is the first pass at the back-end code for account deletion. The text is still rough - I deviated from the draft wireframes slightly to make things easier from a coding perspective for now. I also still need to look into separating pseuds with commas if they're in a list.

There are so many different options involved in the deletion process that I think it could do with quite vigorous testing. Some things to check:

I've done my best to test the above locally and it seems to be working for me, but it could do with checking.

When a work is deleted, currently only a notification is emailed to the user, not a copy of the work itself. Do we want the work to be emailed automatically, or should the user be asked whether they'd like to receive a copy or not?

otwstephanie commented 10 years ago

From highlander.ii@gmail.com on September 20, 2008 17:30:23

I've created a preliminary test plan file for this, since there are so many pieces that will need to be tested. This document doesn't contain ALL of the features that will need to be tested, but has everything that Sarky listed above.

As things are added to this issue, I will try to keep them in the test plan file with 'desired results' listed to make testing this issue easier.

If anything I have listed is awkward or just wrong, let me know and I will update.

My thoughts on this are - since this will be 'removing' items from the database with this particular function, we will want extensive testing to ensure things aren't 'lost' forever. Though, we will have the archive back-ups, but depending on how often those are created, works and data may still be lost.

Attachment: OTW - test plan-pseud deletion.xls

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on September 20, 2008 17:34:19

Yay both of you for being so, so organised!

Yeah, this is complicated - I've got someone lined up to test this specifically so we can work through all the scenarios and they will be doing this today if we can have it applied to the TestArchive.

Labels: Component-Application Milestone-Beta

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 21, 2008 15:19:34

Highlander, you're awesome! cheers on The test plan looks good to me - I think my only comment would be that the tester /might/ need to break Case 8 down a bit further, as there are 6 different form options to be tested there, but you're in a better position to judge that than me. :D

And I entirely agree about the need for extensive testing - that's great that you have someone lined up to work through it, Samvara. Yay for awesome testers! loves

-Cal

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on September 22, 2008 07:53:04

OK, this is a thing of great beauty, almost all of it is perfect which for a first run is impressive!

  1. Delete account page has no "Cancel" button on it.
  2. Radio buttons do not default to option 1.

Otherwise = Awesome!

Attached screen dump with corrections and test scripts

Status: Assigned

Attachment: Delete account.PNG OTW - test plan-pseud deletion v2.xls

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 22, 2008 12:14:42

Yay yay yay! \0 \0/ 0// Revision 824 should add the cancel button and make the radio buttons default to option 1. :D

The front end for the two views still needs to be done in its entirety - apologies to Lim for the non-existent html! They're in the users directory: _delete_preview.html.erb and delete_confirmation.html.erb.

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 22, 2008 12:38:01

Revision 825 should format the lists of pseuds and coauthors more nicely, using commas to separate them.

I've just realised that drafts are being counted towards a user's works on the delete preview page. How should drafts be treated - the same way as a posted work, or differently?

otwstephanie commented 10 years ago

From sarkymoo...@gmail.com on September 22, 2008 14:15:06

Revision 826 has some changes so that drafts are not included in the numbers of works listed on the delete preview form; it is now assumed that they are to be deleted and this should happen automatically when the account is deleted.

Some things I would appreciate the lovely testers checking on here:

otwstephanie commented 10 years ago

From mahar...@yahoo.com on September 24, 2008 06:06:37

Hi,

I tested the above scenarios, and they were all successful. See attached doc for nitty-gritty details :)

Cheers Maharetr

Attachment: OTW - test plan-pseud deletion.xls

otwstephanie commented 10 years ago

From mahar...@yahoo.com on September 24, 2008 06:07:20

Marking as fixed and verified.

Status: FixedAndVerified

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on February 24, 2009 17:43:44

Updating status options

Status: DeployedToBeta

otwstephanie commented 10 years ago

From Black0Sa...@gmail.com on February 24, 2009 19:22:50

Updating status on discussion with Jennifew and Sidra

Status: VerifiedOnBeta

otwstephanie commented 10 years ago

From autu...@gmail.com on November 13, 2010 14:14:03

listing all former issues for automated tests

Status: NeedsTests

otwstephanie commented 10 years ago

From autu...@gmail.com on November 14, 2010 11:40:28

Converting from NeedsTest status to NeedsTest label

Status: VerifiedOnBeta
Labels: NeedsTests