Closed Bug 960047 Opened 6 years ago Closed 6 years ago
Sync banner promo links in bookmark/add-on panels need to continue working; consider re-setting sync counts
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)
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Whiteboard: [qa+] p=0 → p=1 s=it-30c-29a-28b.2 [qa+]
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 https://services.mozilla.com/sync/ 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 https://services.mozilla.com/sync/ 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 https://services.mozilla.com/sync/ 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 https://support.mozilla.org/kb/how-do-i-enable-add-sync Ryan, could you please check which of these wording/links should be changed, or redirect to someone who could?
this is the patch to reset the counters and start showing the promobox again.
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.
Bug 972524 covers sumo, I filed bug 978346 for mozilla.org (which is where the services.mozilla.com 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+]
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.
just an unbitrot
Attachment #8389861 - Flags: review?(gavin.sharp) → review+
https://hg.mozilla.org/integration/fx-team/rev/50734f7aa80d I assume this should get uplifted to Aurora soon.
Target Milestone: --- → Firefox 30
Attachment #8389861 - Flags: approval-mozilla-aurora+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
No longer blocks: fxdesktopbacklog
You need to log in before you can comment on or make changes to this bug.