search

vicolo-dev / chrono

A modern and powerful clock, alarms, timer and stopwatch app for Android!
GNU General Public License v3.0
370 stars 15 forks source link

Timer Notification not updating and Timer Alarm not ringing #190

Open Rafee-M opened 4 months ago

Rafee-M commented 4 months ago

Describe the bug There was in issue with the normal alarm ringing but that has been fixed so far https://github.com/vicolo-dev/chrono/issues/183#issuecomment-2067755993 , but now there is an issue with the Timer alarm not ringing and the notificaion of the Timer also not updating

To Reproduce Steps to reproduce the timer not ringing bug:

  1. Set a Timer for more than 10 minutes
  2. Wait

Steps to reproduce notification bug:

  1. Set a timer
  2. Remove the app from recents menu
  3. Or keep the app in the recents menu and wait
  4. Opening the app normally resumes the stuck notification
  5. Sometimes opening the app from the stuck notification removes it and it comes back after pausing and resuming timer

Expected behavior Timer alarm to ring and notification updating with the timer progress

Screenshots

Intended: The app was just opened to resume stuck timer notification and it was kept in the recents menu

https://github.com/vicolo-dev/chrono/assets/69535896/ff18d3d0-d3ca-44cd-af3b-b9a8d079a7f1

Resuming stuck notification:

https://github.com/vicolo-dev/chrono/assets/69535896/3f9e8723-7784-41e9-8548-2cc4e9a7e5b1

mute audio25 minute timer ringing late by 12 minutes(after opening app): The app is also just a black screen and nothing works (I assume the alarm screen didn't load correctly.

https://github.com/vicolo-dev/chrono/assets/69535896/2f0e4df7-b959-4202-ac55-d477c98a29e2

Smartphone Information

Additional context Comparison (clearing from recents menu) between the notification timer of Simple Clock and Chrono:

https://github.com/vicolo-dev/chrono/assets/69535896/d77670e0-38cb-4956-8179-300e66baa1b5

Update: With the solution mentioned here https://github.com/vicolo-dev/chrono/issues/183#issuecomment-2067755993 timer works properly now. Not sure how it would work for other users without root. Also, the notification bug still exists

AhsanSarwar45 commented 4 months ago

Since the timer uses the same logic behind the scenes as the alarms, yes its going to alsp be unreliable on OnePlus devices if alarms are unreliable. For both these issues I would of course like to find a solution that doesn't require a rooted OnePlus device.

As for the notification, it will only update if the app hasn't been cleared from recents. But either way, it should atleast disappear when app is closed. I might also look at ways of converting it to a background service.

Rafee-M commented 4 months ago

As for the notification, it will only update if the app hasn't been cleared from recents

I see. With Simple Clock it still updates with the app removed from the recents menu but I suppose that's not the design choice here? Also, I'd like to add the even with the app in recents menu, it stops updating after a few minutes or so.

AhsanSarwar45 commented 4 months ago

With Simple Clock it still updates with the app removed from the recents menu but I suppose that's not the design choice here? Also, I'd like to add the even with the app in recents menu, it stops updating after a few minutes or so.

Converting it to a foreground service will probably fix all these issues. Simple Clock probably does the same as its the more proper way to implement this. Will definitely look into this.

Gwiel commented 3 months ago

Unfortunately I have been encountering the bug as well; the alarm only starts ringing once the screen has been activated.

Device: Samsung Galaxy A52S Android 14 App Version: 0.5.1 (latest beta, I think beta 3)

Other than this bug I really love the app design and features :) I hope I can soon make it my primary clock&alarm app

AhsanSarwar45 commented 2 months ago

Related to https://github.com/vicolo-dev/chrono/issues/183#issuecomment-2198205408.