Closed Bug 960047 Opened 11 years ago Closed 11 years ago

Sync banner promo links in bookmark/add-on panels need to continue working; consider re-setting sync counts


(Firefox :: General, defect, P2)




Firefox 30
Tracking Status
firefox29 + verified
firefox30 --- verified


(Reporter: Gavin, Assigned: mak)



(Whiteboard: p=1 s=it-30c-29a-28b.3 [qa!])


(1 file, 1 obsolete file)

Two concerns raised by Marco in bug 958300 comment 1:

- We have "Hey, set up sync" promo banners in the "add bookmark" and "remember password" panels (see bug 618913), those should continue to work with the new sync setup flow

- We only show those panels 5 times (and allow users to explicitly close them to prevent them from hiding them again). We may want to consider re-setting the "don't show anymore" prefs on upgrade to new-sync to re-promote the feature?
we also show the promobox when installing add-ons (if add-ons sync is not enabled)
Whiteboard: [qa+]
Whiteboard: [qa+] → [qa+] p=0
Assignee: nobody → mak77
Whiteboard: [qa+] p=0 → p=1 s=it-30c-29a-28b.2 [qa+]
QA Contact: twalker
So, first of all, these are the links we have in the promo banners, they should be verified and eventually corrected if they are not what we expect, please let me know if we should change some of the interactions here.

Appears on the "Remember Password for this site" doorhanger if Sync is disabled:
- You can access your passwords on all your devices with Sync. Learn More
  clicking Learn More goes to

Appears on the Star panel when editing an already starred bookmark if Sync is disabled:
- You can access your bookmarks on all your devices with Sync. Learn More
  clicking Learn More goes to

Appears on the install add-on doorhanger if Sync is disabled:
- You can access your add-ons on all your devices with %S. Learn More
  clicking Learn More goes to

Appears on the install add-on doorhanger if Sync is enabled but add-ons are not synced:
- You can use your Sync account to synchronize add-ons across multiple devices. Learn More
  clicking Learn More goes to

Ryan, could you please check which of these wording/links should be changed, or redirect to someone who could?
Flags: needinfo?(rfeeley)
Attached patch patch v1 (obsolete) — Splinter Review
this is the patch to reset the counters and start showing the promobox again.
Priority: -- → P2
John, could you please check comment 2 and let me know which of that wording/link I should change?
Flags: needinfo?(rfeeley) → needinfo?(jgruen)
Too late to change the wording for 29, but thankfully those look OK. We're going to need to adjust the links probably, but let's not block this bug on that. I'll file a separate bug to track it.
Flags: needinfo?(jgruen)
Bug 972524 covers sumo, I filed bug 978346 for (which is where the links currently redirect to).
Carry over to Iteration it-30c-29a-28b.3
Whiteboard: p=1 s=it-30c-29a-28b.2 [qa+] → p=1 s=it-30c-29a-28b.3 [qa+]
Depends on: 977472, 972524
Whiteboard: p=1 s=it-30c-29a-28b.3 [qa+] → p=1 [qa+]
Comment on attachment 8381495 [details] [diff] [review]
patch v1

I think we won't need to change the links after all. Let's get this patch landed and then resolve this.
OK, the patch will need an unbitrot since I see someone already added migration 21. Will do that asap.
Attached patch patch v1Splinter Review
just an unbitrot
Attachment #8381495 - Attachment is obsolete: true
Attachment #8389861 - Flags: review?(
Attachment #8389861 - Flags: review?( → review+

I assume this should get uplifted to Aurora soon.
Target Milestone: --- → Firefox 30
Attachment #8389861 - Flags: approval-mozilla-aurora+
Whiteboard: p=1 [qa+] → p=1 s=it-30c-29a-28b.3 [qa+]
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: p=1 s=it-30c-29a-28b.3 [qa+] → p=1 s=it-30c-29a-28b.3 [qa!]
No longer blocks: fxdesktopbacklog
Flags: firefox-backlog+
You need to log in before you can comment on or make changes to this bug.