Closed
Bug 987468
Opened 11 years ago
Closed 11 years ago
Remove "sync is delayed" notification.
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
Firefox 31
People
(Reporter: markh, Assigned: markh)
References
Details
Attachments
(1 file)
7.79 KB,
patch
|
ttaubert
:
review+
rnewman
:
feedback+
|
Details | Diff | Splinter Review |
If sync is delayed, it will show a notification bar with the following strings:
error.sync.no_node_found = The Sync server is a little busy right now, but you don't need to do anything about it. We'll start syncing your data as soon as we can!
29 error.sync.no_node_found.title = Sync Delay
Given we only want to display notifications when user action can resolve the issue, we should remove this notification.
(See also bug 979432, but it's not immediately clear which specific notification it is referring to. Bug 985145 also mentions an unnecessary notification, but that's not referring to this one)
Comment 1•11 years ago
|
||
I'm using bug 979432 as a general tracker for "fix annoying sync notifications".
Blocks: 979432
Assignee | ||
Comment 2•11 years ago
|
||
This removes all "sync was delayed" notifications. It also removes the strings and the sending of the weave:service:sync:delayed notification (hence feedback request on rnewman).
Try at https://tbpl.mozilla.org/?tree=Try&rev=780bd30266d3
Attachment #8398338 -
Flags: review?(ttaubert)
Attachment #8398338 -
Flags: feedback?(rnewman)
Comment 3•11 years ago
|
||
Comment on attachment 8398338 [details] [diff] [review]
0001-Bug-987468-remove-sync-delayed-notification.-r-ttaub.patch
Review of attachment 8398338 [details] [diff] [review]:
-----------------------------------------------------------------
Kill it with fire.
Attachment #8398338 -
Flags: feedback?(rnewman) → feedback+
Updated•11 years ago
|
Attachment #8398338 -
Flags: review?(ttaubert) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Assignee: nobody → mhammond
Status: NEW → ASSIGNED
Comment 5•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 31
You need to log in
before you can comment on or make changes to this bug.
Description
•