Page MenuHomePhabricator

[Notifications] Add first time user Explore feed information card
Open, LowPublic

Description

  • Show a feature education card in the Explore feed to educate the user about the new notification feature.
  • Remove old Explore feed notifications card (see screenshot here)

Product/Design Needs:
[1] What's the expected logic for displaying this in the explore feed?
This card should be shown after first app load after updating the app. This does not need to be shown to first time loads for app downloads.
For more details please see the 'Contributor logs into the app for the first time after Notification update' flow on Figjam

[2] What are the expected actions when tapping a. the card itself b. the got it button and c. the learn more button?
Updated mocks show a 'Turn on push notifications' button and a 'dismiss' link. Tapping 'Turn on push notifications' will trigger the in app model for push permissions and then follow the flow for 'Contributor logs into the app for the first time after Notification update'. Dismiss, dismisses this card from the Explore feed.


Designs
iPhone mockClear background PDF image for use on iPhone or iPad
image.png (1×750 px, 134 KB)

Design details
If possible, can we utilize the card color for the background color for the image so that it translates across reading themes?

Dependencies

https://phabricator.wikimedia.org/T288688

Event Timeline

LGoto lowered the priority of this task from Medium to Low.Aug 17 2021, 6:53 PM

Updated mocks show a 'Turn on push notifications' button and a 'dismiss' link. Tapping 'Turn on push notifications' will trigger the in app model for push permissions and then follow the flow for 'Contributor logs into the app for the first time after Notification update'.

@cmadeo

Sorry, I'm just now looking at this. I've got some new error cases that need to be handled on this one, but even in the success case I'm not certain what will happen. Can you clarify?

  1. Success: User taps "Turn on push" > permissions prompt appears > User allows permissions (behind the scenes we'll also subscribe their device token, that succeeds) > Card hides and user stays on Explore feed? Or user is pushed to Notification Center?
  2. Failure: User taps "Turn on push" > permissions prompt appears > User denies permissions > ?
  3. Failure: User taps "Turn on push" > permissions prompt appears > User allows permissions (behind the scenes we'll also need to subscribe their device token, but that fails) > ?
  4. Failure: User is logged out, so we cannot subscribe their device token behind the scenes.

Suggestion for the failure cases 2 & 3 is that we have some sort of prompt saying there was an issue (missing permissions or issue with subscription), and they can try again later in our Notification Push Settings screen since we're handling all possible error cases there. Then hide the card. Otherwise I think it'll be difficult to handle these error cases within this tiny card.

For failure case 4 I suggest we hide the card if they aren't logged in.

Hi @Tsevener, as discussed in our meeting tapping on the CTA will bring users to the Settings screen and then pop the appropriate alerts / models.

image.png (1×8 px, 638 KB)

Notes from grooming:

This task's explore feed card is meant to be new, as a new feature announcement. We also have an old notifications card that was only meant for the In the News notification, so we need to remove this as a part of this task.

IMG_3966.PNG (2×1 px, 371 KB)

Notes from eng sync: first check existing card show logic, we maybe be able to keep it and simply redesign/reroute if it mostly does what we want already.