search

Audentio / community-app-issues

2 stars 0 forks source link

fix android notifications deeplinking #215

Open tushar-singh opened 4 years ago

tushar-singh commented 4 years ago

┆Issue is synchronized with this ClickUp by Unito

grantrygh commented 4 years ago

Foreground notifications should be working now (correct snackbar popup link & following system notifications correctly shows webview if app is open)

Still need to look into notifications when the app is closed. Current behavior opens the app, but doesn't navigate to the notification url in the webview

grantrygh commented 4 years ago

Haven't been able to figure out the handling of the push notifications to correctly navigate to the specified webview url when the app is closed on android yet.

grantrygh commented 4 years ago

@kylerc tagging you on this in case you're able to take a look at some point - know you're busy with a bunch of stuff

not sure if this is related or not : https://github.com/expo/expo/issues/6943

thought it was, but everything seems to function correctly on iOS for us, just android issues

kylerc commented 4 years ago

Seems like that might be the case. You might try the android suggestion at https://github.com/expo/expo/issues/6943#issuecomment-675135421 and see if that helps

grantrygh commented 4 years ago

the mentioned 'fix' doesn't seem to fix anything. will try again when I get the time, but that expo issue was just closed yesterday

kylerc commented 4 years ago

https://github.com/sjchmiela/expo-notifications-initial-notification-test/blob/master/App.tsx they have this as an example, does this work any better @grantrygh ?

grantrygh commented 4 years ago

@kylerc I've tried that as well, no luck

kylerc commented 4 years ago

Hmm, I think this looks like a bug with expo that sounds like it'll be fixed in expo 39 in September so going to ice box until then.

grantrygh commented 4 years ago

Tested this a bit with expo 39 and still not able to get this working