ACINQ / phoenix

Phoenix is a self-custodial Bitcoin wallet using Lightning to send/receive payments.
https://phoenix.acinq.co
Apache License 2.0
644 stars 97 forks source link

Background app refresh: outdated information #341

Closed robbiehanson closed 1 year ago

robbiehanson commented 1 year ago

Now that we have transitioned to using a notification-service-extension, the information surrounding background app refresh is a little bit outdated.

For example, we say that "Background app refresh" must be enabled to receive payments in the background. This was correct when using "silent push notifications", but is technically incorrect with a notification-service-extension. However, we're currently forcing it to be correct by unregistering for push notifications, based on the "background app refresh" status.

However, the watch tower task DOES require background app refresh. So if the user disables it, we need to warn them somewhere within the app.