Include an onboarding slide to ask for notification permissions

RESOLVED WONTFIX

Status

()

Firefox for iOS
General
RESOLVED WONTFIX
4 months ago
4 months ago

People

(Reporter: st3fan, Assigned: st3fan)

Tracking

unspecified
Other
iOS

Firefox Tracking Flags

(fxios?)

Details

(Whiteboard: [MMA][Onboarding][Strings])

Attachments

(6 attachments)

(Assignee)

Description

4 months ago
Include an onboarding slide to ask for notification permissions. This should only show for the same audience as Leanplum, and then only for the english speaking segment.

This needs to be added to the v8.x onboarding, so it needs to be in the old style.

Screenshot attached. Note that the screenshot 

Does it need to emit leanplum events or attributes so that we know what the opt-in rate is to this screen?
(Assignee)

Comment 1

4 months ago
Created attachment 8896074 [details]
Screenshot 2017-08-10 22.03.08.png
(Assignee)

Updated

4 months ago
Whiteboard: [MMA][Onboarding]
Please could I have an asset for this?

Currently using as placeholder: https://github.com/mozilla-mobile/firefox-ios/blob/master/Client/Frontend/Intro/Intro.xcassets/tour-Private.imageset/tour-Private%402x.png
Flags: needinfo?(randersen)
Assignee: nobody → jhugman
(Assignee)

Comment 3

4 months ago
Created attachment 8896413 [details] [review]
PR https://github.com/mozilla-mobile/firefox-ios/pull/3031

Work in progress patch.
Created attachment 8896415 [details]
tourPushNotif-v8.png

push notif asset
Flags: needinfo?(randersen)
Created attachment 8896417 [details]
Interstitial 8.x.png

In context. Use gold bgcolor from first slide and same type/button styles.
Created attachment 8896439 [details]
tourPushNotif-v8.zip

Er, ahem, all sizes of 8.x asset.
Created attachment 8896682 [details]
Transparent assets!

/me hangs head in shame
(Assignee)

Updated

4 months ago
Assignee: jhugman → sarentz
(Assignee)

Updated

4 months ago
Attachment #8896413 - Flags: ui-review?(randersen)
Attachment #8896413 - Flags: review?(jhugman)
Comment on attachment 8896413 [details] [review]
PR https://github.com/mozilla-mobile/firefox-ios/pull/3031

This looks good. A few nits.
Attachment #8896413 - Flags: review?(jhugman) → review+
(Assignee)

Updated

4 months ago
Whiteboard: [MMA][Onboarding] → [MMA][Onboarding][Strings]
(Assignee)

Updated

4 months ago
Whiteboard: [MMA][Onboarding][Strings] → [MMA][Onboarding]
(Assignee)

Updated

4 months ago
Whiteboard: [MMA][Onboarding] → [MMA][Onboarding][Strings]
(Assignee)

Updated

4 months ago
tracking-fxios: 8.2 → ?
Attachment #8896413 - Flags: ui-review?(randersen) → ui-review+

Comment 9

4 months ago
1 failures in 949 pushes (0.001 failures/push) were associated with this bug in the last 7 days.   

Repository breakdown:
* autoland: 1

Platform breakdown:
* osx-10-10: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1389355&startday=2017-08-14&endday=2017-08-20&tree=all

Updated

4 months ago
Status: NEW → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.