search

pmusaraj / discourse-mobile-single-site-app

Single-site app wrapper for a Discourse site. Supports Push Notifications via OneSignal.
MIT License
81 stars 24 forks source link

Major: Android users logged out after a few minutes #10

Closed jorgtron closed 6 years ago

jorgtron commented 6 years ago

To replicate: leave app open for 5-ish minutes on android device and close app. You'll have to log in again on next open. Reported by multiple users, but I haven't seen it myself.

pmusaraj commented 6 years ago

I could not reproduce this on an emulator. I loaded the app after a few weeks of not running it, and it logged back in to the site correctly.

My emulator is running API 27. It's possible this is an issue with a specific version/flavour of Android.

I will be able to test tomorrow with a real device (older than API 27) and report back.

It would be useful to have more information on the devices of the users that have seen this problem, specifically the Android version on each device.

jorgtron commented 6 years ago

I have these reports so far:

pmusaraj commented 6 years ago

Ok, I can reproduce now on Android 8 (API 26) emulator.

pmusaraj commented 6 years ago

Alright, I think I found the issue. Users do not get logged out, but the app doesn't correctly store the state in some cases, which means that users will see the initial login screen (even if they are actually logged in to the app).

This is now fixed, I think. And current users can click on "Skip" and this will show their current session and fix the issue for further sessions.

Thanks!