Closed Bug 1089762 Opened 10 years ago Closed 10 years ago

add text link to sms option to android landing page

Categories

(www.mozilla.org :: Pages & Content, defect)

Production
x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jbertsch, Assigned: jpetto)

References

Details

(Whiteboard: [kb=1556999] )

Attachments

(1 file)

Attached file sms_update.pdf
Add "Text a download link to my Android phone" link below download button on Android page for all desktop (non-mobile) users. pointing to https://www.mozilla.org/en-US/firefox/sms/ Can we wrap this in the Nov 10 work? :)
Blocks: 1077130
Assignee: nobody → jon
Whiteboard: [kb=1556999]
Jon, this is a nice-to-have for Nov 10, not a blocker.
Adding the link isn't a problem, but localization could be. Maybe We could go en-US only at first behind an L10n tag? Adding flod & pascalc to the bug for comment.
Actually... Winston do we need to add geolocation to this request due to the data charges? Also, is en-US only ok?
Flags: needinfo?(wbowden)
Hey there - this should only be shown to users in the U.S. The SMS service won't work for non-US users. If the pilot succeeds, we will roll this out to additional markets.
Flags: needinfo?(wbowden)
jpetto - how much scope does that add? :)
None! I can just wrap the link in a conditional checking for the en-US locale. Now we don't have to additionally burden the localizes. It's win-win.
Is there any reason this shouldn't go out with the Android push on 10/29?
(In reply to Jon Petto [:jpetto] from comment #7) > Is there any reason this shouldn't go out with the Android push on 10/29? No reason at all! If we could get this in for 10/29, that would be beyond fabulous. Thanks!
(In reply to Jon Petto [:jpetto] from comment #6) > None! I can just wrap the link in a conditional checking for the en-US > locale. Now we don't have to additionally burden the localizes. It's win-win. I think the majority of en-US users are outside of the US, so filtering by locale is a good first step but I think you will need geolocation too.
Added to the PR: https://github.com/mozilla/bedrock/pull/2388/commits Based on comment 9, should geolocation be a blocker here?
(In reply to Jon Petto [:jpetto] from comment #10) > Added to the PR: https://github.com/mozilla/bedrock/pull/2388/commits > > Based on comment 9, should geolocation be a blocker here? Yes please.
Ok. I don't think it will be too hard to add geolocation. Will update the PR.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: