Closed Bug 468691 Opened 16 years ago Closed 15 years ago

Need UI for when something is wrong

Categories

(Cloud Services :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: hello, Assigned: Mardak)

References

Details

Attachments

(2 files, 3 obsolete files)

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.
Assignee: nobody → edilee
Bug 480057 will implement the various resets for the UI to call.
Depends on: 480057
Blocks: 480057
No longer depends on: 480057
Depends on: 482792
Attached image screenshot of v1 data pane (obsolete) —
Attached image screenshot of v1 advanced pane (obsolete) —
Attached patch v1 (obsolete) — Splinter Review
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.
Attachment #366903 - Flags: review?(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.
(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.. (?)
(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?
Sure. Relabel them as "Wipe Local Data", "Wipe Remote Data" and the "Reset Server Data" to be "Wipe Server Data" ?
Oh and under Debugging Tools?
(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).
5) reset all (for restore from backup, or make a new profile by copy information, but not settings, from another broken profile or pc)
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.
(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.
Attached image screenshot of v2
Moved the new buttons to Advanced/Debug. Left the data pane untouched.
Attachment #366895 - Attachment is obsolete: true
Attachment #366897 - Attachment is obsolete: true
Attached patch v2Splinter Review
4 new buttons. I could probably refactor a bunch of this...
Attachment #366903 - Attachment is obsolete: true
Attachment #366960 - Flags: review?(thunder)
Attachment #366903 - Flags: review?(thunder)
Attachment #366960 - Flags: review?(thunder) → review+
Comment on attachment 366960 [details] [diff] [review]
v2

r+, but noting that this is not final UI.  This bug should remain open for that.
Initial (debug) UI:
http://hg.mozilla.org/labs/weave/rev/67818b5bd58b
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Priority: -- → P1
Version: unspecified → Trunk
Blocks: 482900
(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?
Blocks: 482902
Component: Weave → General
Product: Mozilla Labs → Weave
Version: Trunk → unspecified
QA Contact: weave → general
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: