tughi / aggregator-android

News reader for Android
https://tughi.github.io/aggregator-android
GNU General Public License v3.0
66 stars 8 forks source link

[Bug] App launch isn't always recognized, preventing feed update on application launch #76

Closed RequestPrivacy closed 1 year ago

RequestPrivacy commented 1 year ago

Describe the bug After quitting the app and reopening the feeds aren't updated even though update mode is set "Auto update: on app launch"

To Reproduce

NOW

Reasoning I am picky about how often and at what times my device connects to the internet. I want to be in control. Therefore I tend to limit background updates/refreshes. "Update on app launch" seems a good fit for me. Yet I'm under the impression that Aggregator can't differentiate to well between a short therm switch to/from the app and a "new" app launch unless you really kill the app manually via the task manager?

It would really help to have a manual way to update e.g. via a pull down action.

BTW, great app, many thanks for your work!

Device (please complete the following information):

Screenshots Not applicable

tughi commented 1 year ago

Hmmm... I can confirm that using the new gestures, Android doesn't finish the MainActivity anymore when you use the "back gesture". This is the reason why app-launches aren't recongnized anymore untill Android kills itself the app.

As a workaround the MainActivity has to use an OnBackPressedDispatcher to recognize the user's intention to leave the app and prepare for a new app-launch.

RequestPrivacy commented 1 year ago

Thanks for fixing. Looking forward for F-Droid providing the new version in their repos...