468691 - Need UI for when something is wrong

Status: RESOLVED FIXED

(Cloud Services :: General, defect, P1)

Description

[Dan Mills [:thunder]]

We need to think about what UI is needed (on both Firefox and Fennec) for when the user thinks something data is not being synced correctly.

Comment 1

[Ed Lee :Mardak]

Bug 480057 will implement the various resets for the UI to call.

Comment 2

[Ed Lee :Mardak]

Attachment: screenshot of v1 data pane

Comment 3

[Ed Lee :Mardak]

Attachment: screenshot of v1 advanced pane

Comment 4

[Ed Lee :Mardak]

Attachment: v1

For now, we can just have buttons that wipes everything locally/remotely from the Data pane. Also we can provide the wipe server button on the advanced page with the other server stuff.

I called wiping remote/local stuff as "initialize".. kinda like mac's initialize for reformatting a drive.

Comment 5

[Dan Mills [:thunder]]

(In reply to comment #4)
> Created an attachment (id=366903) [details]
> v1
> 
> For now, we can just have buttons that wipes everything locally/remotely from
> the Data pane. Also we can provide the wipe server button on the advanced page
> with the other server stuff.

Is this meant for testing while we build something like this?

https://bugzilla.mozilla.org/show_bug.cgi?id=480057#c6

> I called wiping remote/local stuff as "initialize".. kinda like mac's
> initialize for reformatting a drive.

OSX's Disk Utility says "Erase", FWIW.  I do remember seeing "initialize" somewhere, but not sure where.

I think the terminology we came up with before, "wipe" & "reset", is OK.

Comment 6

[Ed Lee :Mardak]

(In reply to comment #5)
> Is this meant for testing while we build something like this?
> https://bugzilla.mozilla.org/show_bug.cgi?id=480057#c6
Right. Getting the finer grained wiping ability would require a bit more work, but just wiping everything should be "good enough" for a first pass.. (?)

Comment 7

[Dan Mills [:thunder]]

(In reply to comment #6)
> (In reply to comment #5)
> > Is this meant for testing while we build something like this?
> > https://bugzilla.mozilla.org/show_bug.cgi?id=480057#c6
> Right. Getting the finer grained wiping ability would require a bit more work,
> but just wiping everything should be "good enough" for a first pass.. (?)

Ok, can we move the buttons to the advanced pane then?

Comment 8

[Ed Lee :Mardak]

Sure. Relabel them as "Wipe Local Data", "Wipe Remote Data" and the "Reset Server Data" to be "Wipe Server Data" ?

Comment 9

[Ed Lee :Mardak]

Oh and under Debugging Tools?

Comment 10

[Dan Mills [:thunder]]

(In reply to comment #9)
> Oh and under Debugging Tools?

Yeah.

What's the difference between Wipe Remote Data, Wipe Server Data, and Reset Server Data?

I think you want 3-4 things:

1) Wipe local client and sync down server data
2) Wipe server data (and potentially also other clients) and upload local data
3) Cause a full 2-way sync again (reset all engines)

And optionally

4) Wipe or reset other clients (if you want to test that separately from a server wipe)

These will map pretty cleanly to the UI on bug 480057 (left arrow, right arrow, 2-way arrow).

Comment 11

[Igor Velkov]

5) reset all (for restore from backup, or make a new profile by copy information, but not settings, from another broken profile or pc)

Comment 12

[Ed Lee :Mardak]

Just to be clear, there's 4 buttons in the patch:
"Force Sync Now..."
"Initialize Local Data..."
"Initialize Remote Data..."
"Reset Server Data"

I suppose "Initialize" is a better fit than just plain "Wipe" because the action actually wipes local data AND re-syncs.

The 4 buttons in the patch do provide the functionality you described.

(1) wipe local client -> Initialize Local Data
(2) wipe server -> Reset Server Data
(3) 2-way sync -> Force Sync Now
(4) wipe other clients -> Initialize Remote Data

The only slight difference is for (2), you say to upload local data.. right now the patch just does a wipe, but it can easily force a sync after.

Comment 13

[Dan Mills [:thunder]]

(In reply to comment #11)
> 5) reset all (for restore from backup, or make a new profile by copy
> information, but not settings, from another broken profile or pc)

That's outside of the scope of this bug.

(In reply to comment #12)

> I suppose "Initialize" is a better fit than just plain "Wipe" because the
> action actually wipes local data AND re-syncs.

Initialize sounds way too harmless, though.  You're going to be erasing data, so I want a scary word.

> The 4 buttons in the patch do provide the functionality you described.
> 
> (1) wipe local client -> Initialize Local Data
> (2) wipe server -> Reset Server Data
> (3) 2-way sync -> Force Sync Now
> (4) wipe other clients -> Initialize Remote Data

Perhaps "erase" is what we want:

(1) Erase Local Data
(2) Erase Server Data
(3) Reset Sync Metadata
(4) Erase Other Clients

And I think this UI should only appear if some about:config pref is set.

No need to cause an immediate sync after 1, 2, or 3.  I think you need it for 4, because of how it works.

Comment 14

[Ed Lee :Mardak]

Attachment: screenshot of v2

Moved the new buttons to Advanced/Debug. Left the data pane untouched.

Comment 15

[Ed Lee :Mardak]

Attachment: v2

4 new buttons. I could probably refactor a bunch of this...

Comment 16

[Dan Mills [:thunder]]

Comment on attachment 366960 [details] [diff] [review]
v2

r+, but noting that this is not final UI.  This bug should remain open for that.

Comment 17

[Ed Lee :Mardak]

Initial (debug) UI:
http://hg.mozilla.org/labs/weave/rev/67818b5bd58b

Comment 18

[Dan Mills [:thunder]]

(In reply to comment #0)
> We need to think about what UI is needed (on both Firefox and Fennec) for when
> the user thinks something data is not being synced correctly.

We didn't really get to that.  Seems this bug should stay open?