1389355 - Include an onboarding slide to ask for notification permissions

Status: RESOLVED WONTFIX

(Firefox for iOS :: General, enhancement)

Description

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

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?

Comment 1

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Attachment: Screenshot 2017-08-10 22.03.08.png

Comment 2

[James Hugman [:jhugman] [@jhugman]]

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

Comment 3

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Attachment: PR https://github.com/mozilla-mobile/firefox-ios/pull/3031

Work in progress patch.

Comment 4

[Robin Andersen [:tecgirl]]

Attachment: tourPushNotif-v8.png

push notif asset

Comment 5

[Robin Andersen [:tecgirl]]

Attachment: Interstitial 8.x.png

In context. Use gold bgcolor from first slide and same type/button styles.

Comment 6

[Robin Andersen [:tecgirl]]

Attachment: tourPushNotif-v8.zip

Er, ahem, all sizes of 8.x asset.

Comment 7

[Robin Andersen [:tecgirl]]

Attachment: Transparent assets!

/me hangs head in shame

Comment 8

[James Hugman [:jhugman] [@jhugman]]

Comment on attachment 8896413 [details] [review]
PR https://github.com/mozilla-mobile/firefox-ios/pull/3031

This looks good. A few nits.

Comment 9

[Intermittent Failures Robot]

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