Audentio / community-app-issues

2 stars 0 forks source link

fix android notifications deeplinking #788

Open sync-by-unito[bot] opened 3 years ago

sync-by-unito[bot] commented 3 years ago

┆Issue is synchronized with this ClickUp by Unito

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

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

sync-by-unito[bot] commented 3 years ago

➤ Kyler Carlson commented:

➤ Kyler Carlson commented:

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.

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

kylerc I've tried that as well, no luck

sync-by-unito[bot] commented 3 years ago

➤ Kyler Carlson commented:

➤ Kyler Carlson commented:

https://github.com/sjchmiela/expo-notifications-initial-notification-test/blob/master/App.tsx ( 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 ?

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

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

sync-by-unito[bot] commented 3 years ago

➤ Kyler Carlson commented:

➤ Kyler Carlson commented:

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

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

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 ( https://github.com/expo/expo/issues/6943 )

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

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

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.

sync-by-unito[bot] commented 3 years ago

➤ Grant Rygh commented:

➤ Grant Rygh commented:

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