Closed Bug 1261172 Opened 8 years ago Closed 8 years ago

Start using Sync link in Dev Edition General Prefs links to deprecated Sync page

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 48
Tracking Status
firefox48 --- fixed

People

(Reporter: rfeeley, Assigned: tcsc)

Details

Attachments

(1 file)

The “Start using Sync…” link in Dev Edition under General Prefs links to a deprecated Sync page about:accounts.

It should instead link directly to this signin page.

about:accounts?action=signin&entrypoint=menupanel

We should give it a proper entry point name though. NIing RFkelly for that.
Flags: needinfo?(rfkelly)
I don't think we have a terribly consistent naming scheme for these, but how about:

   about:accounts?action=signin&entrypoint=dev-edition-setup
Flags: needinfo?(rfkelly)
SGTM and looks like a trivial change at https://dxr.mozilla.org/mozilla-central/source/browser/components/preferences/in-content/main.js#256
Component: Preferences → Sync
Flags: firefox-backlog+
Assignee: rfkelly → tchiovoloni
Status: NEW → ASSIGNED
Comment on attachment 8738010 [details]
MozReview Request: Bug 1261172 - update link to sync sign-in page shown in dev edition r?markh

https://reviewboard.mozilla.org/r/44251/#review40915
Attachment #8738010 - Flags: review?(markh) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/451cde555145
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: