pmusaraj / discourse-mobile-single-site-app

Single-site app wrapper for a Discourse site. Supports Push Notifications via OneSignal.
MIT License
81 stars 24 forks source link

Push notification redirects in app #13

Closed jorgtron closed 6 years ago

jorgtron commented 6 years ago

If the user received a push notification while the app is open, there is only an option to select "OK" on the notifcation. When the user taps "OK" he is taken to the URL for the notification. This can be annoying if the user is reading a topic or composing a post, because they can't stop the redirect. Adding another option to the modal could perhaps work, for example having "Show" or "Close" as the options instead of "OK"

jorgtron commented 6 years ago

I've seen this happen and when redirected the new post I was writing was lost - very frustrating.

pmusaraj commented 6 years ago

Thanks, fixed as of this commit: https://github.com/pmusaraj/discourse-mobile-single-site-app/commit/c98ab1468ffb03030ff9793d17fe43af99d995a6