search

bobness / inspect-app

0 stars 0 forks source link

Tapping `Create Summary` button creates it in the background, but does not close the `SummaryScreen` #33

Closed bobness closed 1 year ago

bobness commented 1 year ago

It also does not notify followers. Related? maybe

bobness commented 1 year ago

OK, I added debug statements in submitShare() and confirmed that it posts the new summary, but that sendNotification() never returns -- in inspect-service's database it would appear it's because there are no expo_token values for either user.

So apparently they are not being updated as expected still.

bobness commented 1 year ago

Oh, I see. It would appear that the Device.isDevice in registerForPushNotificationsAsync() assumes that the device creating the summary is also a real device. Not just the one receiving. I tried commenting out that check, but it gives an error that UIBackgroundModes is not configured in the Info.plist file...

bobness commented 1 year ago

But @dreitec did it work for you without that plist property? Maybe that's going in wrong direction. Maybe I need to have another device to create a summary in real time so the other device gets a notification?

bobness commented 1 year ago

This is fixed -- the issue was that sendNotification wasn't returning because of a typo in notification.js that I introduced: https://github.com/bobness/inspect-service/commit/16341110cc71eae3efb15147057b9a27496880ce