Closed Bug 758593 Opened 14 years ago Closed 14 years ago

Wipe Data does not send logout message

Categories

(Pancake Graveyard :: Front-end, defect)

Product:
Pancake Graveyard
Component:
Front-end
Platform:
x86
macOS
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: st3fan, Assigned: st3fan)

Details

(Keywords: regression)

Wipe Data from the account menu does not send a logout message to the native code anymore. I do see it POST to /api/wipe correctly, which successfully calls lattice and user-api. It just doesn't notify the iOS code to logout the user.
I've added tests and refactored this to use a promises sequence that in my testing does the right thing: it cancels the whole operation if the user hits cancel on the confirm dialog, it sends a logout message if the /wipe/ request is successful, and doesn't if it is not. I wasn't able to reliably reproduce this issue, but one of the things I spotted and fixed along the way was that api/wipe will send a 200 response with { success: false } if the wipe wasn't successful for whatever reason. That was previously triggering the 'wipe successful' code path. Now we have an ajax adapter in place that pushes all such responses down the error path. See commits: https://bitbucket.org/mozillapancake/pancake/changeset/6bb0e6239293 .. thru' https://bitbucket.org/mozillapancake/pancake/changeset/e4635e79e4c3
Can you confirm this is fixed? WFM..
Assignee: sfoster → sarentz
No I am still seeing this. Yesterday when running the app from the simulator on development.
I added a spinner to the account menu UI so its clearer what's going on. For me, the spinner never stops spinning - there's no response (no success, no failure) from the server.
There was a bug in Lattice that would not return a response upon calling /user/wipe. That got fixed. I have /user/wipe working for me correctly now
The account menu "clear data" button does indeed work. We need to do more with the subsequent steps - we delete the user account and redirect back to the initial signin/setup screen. The setup sequence is littered with uninformative errors at this point. But these need their own tickets.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
End Users should get a messaging as well stating that they are being logged out of the application (bug 768728) Application does logout after clearing data Verified 6/26/2012 DEv
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.