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

Push notification does not send me to a place where I can participate #76

Closed
youralien opened this issue Aug 9, 2018 · 2 comments
Closed

Comments

@youralien
Copy link
Member

Setup: keep the app open on the results page, but leave it in the background

Issue: Two old notification messages were on my home screen, so clicking them opened the app, but did not direct me to the pages where experiences are available. I'm not sure why that is (whether the link to the incident is broken), or if we tried to just redirect people to the '/' home route

@youralien youralien changed the title Push notification does not send me to the home screen Push notification does not send me to a place where I can participate Aug 9, 2018
@youralien
Copy link
Member Author

youralien commented Aug 12, 2018

The link to the participate screen was broken

in executor.js

      // should be
      // let route  = `apicustom/${iid}/${incident.eid}/${needName}`
      let route = "apiCustom/" + iid + "/" + needName;
      notifyForParticipating(newUsersUids, iid, "Event " + experience.name + " is starting!",
        experience.notificationText, route);

@youralien
Copy link
Member Author

One hack is to route users to the home page. This requires one extra step to participating. However, it always ensures they see participation screens they are assigned to

  1. If we send them to the participate route, but they are no longer assigned, they shouldn't be able to participate. If they were asked to participate, but no longer are assigned, this route could redirect them to a page that apologizes that they missed the opportunity to participate. This is related to Issue User is available, push is sent, user updates location, assignments are cleared, user is not longer available #46. This would reduce the number of "apology notifications", and would only show this apology when they thought they could participate.

@youralien youralien mentioned this issue Aug 14, 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

1 participant