Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable deep linking to new "Follow" push notification #1628

Closed
4 tasks
zavreb opened this issue Dec 9, 2017 · 9 comments
Closed
4 tasks

Enable deep linking to new "Follow" push notification #1628

zavreb opened this issue Dec 9, 2017 · 9 comments

Comments

@zavreb
Copy link

zavreb commented Dec 9, 2017

Issue: Presently, users aren't aware when another user follows them. To increase traffic to the app, we'd like to notify the user every time another user follows them via push notification (provided they have it enabled)

REQs:

  • Must match mock, see here zeplin: zpl.io/2p0NgxE
  • Fired when another user follows the owner
  • When user taps, it deep links to User Profile (if it's not too hard to deep link into Followers screen see below req.)

Nice-to-have:

  • Deep links to Followers screen for owner

New Follow Push Notification

image

Deep links to one of these screens (right is preferred)

image

@zavreb zavreb added this to the Sprint #21.1 | Dec 8 - Dec 20 milestone Dec 9, 2017
@zavreb
Copy link
Author

zavreb commented Dec 9, 2017

@thescurry to review design.

@thescurry
Copy link

LGTM

@zavreb
Copy link
Author

zavreb commented Dec 12, 2017

@bengtan to create wocky related ticket to create new push notification

@zavreb zavreb changed the title Enable new "Follow" push notification Enable deep linking to new "Follow" push notification Dec 12, 2017
@bengtan
Copy link
Contributor

bengtan commented Dec 12, 2017

Before I write the server side ticket, I need to know:

  • What is the format of the deeplink? This depends on which screen we are linking to (ie. 'right is preferred').

Also, in a more general sense, weren't we going to review push notifications (and app badge) and possibly overhaul them?

@zavreb
Copy link
Author

zavreb commented Dec 12, 2017

Per call: Deep linking pushes new screen over current one. If user is on Followers screen, and taps on push notification, it'll stack a duplicate, if you tap "back" it'll take you to previous Followers page.

@mstidham
Copy link

I need a new follower to test this on Prod

@zavreb
Copy link
Author

zavreb commented Dec 20, 2017

Wilderness testing needed, can't reproduce it.

@thescurry
Copy link

I have never received a "new follower" push notification before... has anyone else? Should we see these with auto follows?

@mstidham
Copy link

mstidham commented Mar 1, 2018

I need a new follower to test this on Prod.

@zavreb zavreb closed this as completed May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants