LeanderBB / you-have-mail

Application to notify you if an email has arrived in your email Account
GNU Affero General Public License v3.0
88 stars 5 forks source link

Not polling for new emails after upgrading from v0.16.2 to v0.16.3 #47

Closed Zaedus closed 2 months ago

Zaedus commented 3 months ago

When updating to the new version I might get one notification if I open the app, but no more after that without manually polling. In the app, it will sometimes have been a whole day since it last polled. Looking at the diff between these versions, a lot of code that handles this has changed. I've opted to downgrade to v0.16.2 since I need to know when I get an email.

Edit: I am running GrapheneOS on a Pixel 7a

julianfairfax commented 3 months ago

I also have this issue on my Pixel 6a running GrapheneOS

LeanderBB commented 3 months ago

Hey, thanks for the report. For me this has been working pretty consistently so far on a Pixel 6 with Graphene OS. I have my poll interval set to 15 min.

What I have noticed is that if I don't touch the phone for a long period of time the phone will enter a doze mode and work scheduled with WorkManager will be delayed by the OS.

@Zaedus Are you sure you don't mean between v0.15 and v0.16.x? Not a lot has changed between 0.16.2 and 0.16.3. In fact if you run 0.16.2 you are more likely to miss the notifications all together.

Could you folks share your logs and your poll intervals?

julianfairfax commented 3 months ago

you-have-mail-logs.zip 5 minutes

LeanderBB commented 3 months ago

@julianfairfax thanks for the logs. Could you try running with a poll interval >= 15 min to see if the experience improves?

There are 2 types of workers currently in use. For anything < 15 min I need to use a different type of worker. If the experience improves, there may be an issue with the shorter poll intervals.

LeanderBB commented 3 months ago

@julianfairfax @Zaedus did you change the time interval when upgrading from 0.15.x to 0.16.x? If not can you try to change it to something else and back to whatever you want. I have the suspicion that there may have been a bug in migrating timer values from the old to the new version.

I have done some quick checks with the 1-5 min intervals and they seem to be working reliably.

julianfairfax commented 3 months ago

@julianfairfax thanks for the logs. Could you try running with a poll interval >= 15 min to see if the experience improves?

There are 2 types of workers currently in use. For anything < 15 min I need to use a different type of worker. If the experience improves, there may be an issue with the shorter poll intervals.

Done. This seems to be working nwo.

@julianfairfax @Zaedus did you change the time interval when upgrading from 0.15.x to 0.16.x? If not can you try to change it to something else and back to whatever you want. I have the suspicion that there may have been a bug in migrating timer values from the old to the new version.

I have done some quick checks with the 1-5 min intervals and they seem to be working reliably.

I did not. I have even uninstalled the app entirely and reinstalled it without success.

something15525 commented 3 months ago

I've been having this issue as well in a Pixel 7 Pro running GrapheneOS, with a polling interval of 5 min. Lemme try 15 min and see if that makes things more reliable.

LeanderBB commented 3 months ago

I have installed the app on a Pixel 3a at 5 min. Will see how it fares during the day against my main phone at 15min.

LeanderBB commented 3 months ago

Confirmed the issue. I recommend everyone stay on the >= 15 min intervals until the fix is out.

LeanderBB commented 3 months ago

This should be addressed in v0.16.4. Let me know if it fixes the issue.

However, do keep in mind that if the phone enter doze mode the poll work will be delayed by the OS to preserve battery life regardless of poll interval.

something15525 commented 2 months ago

Can confirm a 15 min interval fixed if for me. @LeanderBB thanks for the quick fix!

Zaedus commented 2 months ago

Latest version fixed the bug for me!

Edit: I can set it to a 1 minute interval with no issue

LeanderBB commented 2 months ago

Perfect. Thanks for the feedback :).

something15525 commented 2 months ago

Works for me as well, thanks for the fix!