Explain need for notification permission: sent tabs

RESOLVED FIXED

Status

()

Firefox for iOS
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: wesj, Assigned: fluffyemily)

Tracking

unspecified
Other
iOS

Firefox Tracking Flags

(fxios+)

Details

Attachments

(1 attachment)

47 bytes, text/x-github-pull-request
jhugman
: review+
Details | Review | Splinter Review
(Reporter)

Description

3 years ago
I just looked at Fennec and saw it asking for permissions to send me notifications. Without any context as to why it needs that, it was confusing. We should make sure that the user has some idea what's going on before we prompt.
I see this on access to the remote tab pane on first launch. If we're not using notifications we shouldn't prompt.
tracking-fxios: --- → ?
We need this to show you sent tabs. We could request this permission when we set up FxA.
Summary: Notification permission pops up at random times → Explain need for notification permission: sent tabs
(Assignee)

Comment 3

3 years ago
it will request permissions in 2 instances:

1.when you first sign up to an account (and it kicks off it's first sync)
2. if you are already signed into an account when the notification code lands, you will see it when the first sync occurs.

It should not ask you any other time, because unless you are performing syncs then we will never send you notifications
I don't know much about the iOS permission model.  As a user, I would want to:

* be prompted to allow notifications when setting up the account;
* be able to view and change my choice in the future.

Perhaps we can do whatever requests the permission at account set up time?  And I think the choice is managed by iOS, so perhaps the latter is already covered?  Is it possible to inspect the choice and avoid taking whatever action requests the permission when permission has already been denied?

Updated

3 years ago
tracking-fxios: ? → +
Move prompt to when the user signs in so there some more context for v1.0 and we'll revisit this after to provide additional context such as an info screen on what the notifications are for.
(Assignee)

Updated

3 years ago
Assignee: nobody → etoop
(Assignee)

Comment 6

3 years ago
Created attachment 8632730 [details] [review]
Pull request
Attachment #8632730 - Flags: review?(jhugman)
(Assignee)

Updated

3 years ago
Status: NEW → ASSIGNED
Attachment #8632730 - Flags: review?(jhugman) → review+
(Assignee)

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.