Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong calculation for the repeated alarm #831

Closed
islam2hamy opened this issue Sep 21, 2024 · 6 comments
Closed

Wrong calculation for the repeated alarm #831

islam2hamy opened this issue Sep 21, 2024 · 6 comments
Labels

Comments

@islam2hamy
Copy link

Describe the bug
Wrong calculation for the repeated alarm

To Reproduce
Steps to reproduce the behavior:

  1. Set notification alarm , 10m before any event , with repeat every day
  2. wait for the notification , and remove it from the notifications
  3. open the app to see next repeat

Expected behavior
Correct calculation for the next repeated alarm

Version Info:

  • App Version: 0.16.1
  • Android Version: 7.1.2
  • Device Model: Redmi 4x
@forrestguice
Copy link
Owner

The rescheduling part should happen when the notification is dismissed. It seems that isn't happening, so when the app is re-opened it still shows the previously scheduled time.

Does it work correctly if you tap the notification instead of swiping/removing it?

I've noticed another bug while trying to reproduce this one. The foreground service is still running after the notification is shown. The progress message should only appear for a moment, but it sticks around until the notification is dismissed.

@islam2hamy
Copy link
Author

Does it work correctly if you tap the notification instead of swiping/removing it?

No

@forrestguice
Copy link
Owner

hrm, it must be a different problem than the one I'd assumed.
I think I'll need to see a before / after screenshot of the alarm list to understand better.

@islam2hamy
Copy link
Author

islam2hamy commented Sep 23, 2024

adds "use app location" option to alarms; reschedules alarms automatically when the location is changed (#768, #808).

Let me test some thing about DST , as my old phone android 7.1.2 doesn't support it with Egypt, so i have changed my phone location to Istanbul , and the alarm is set to egypt

@islam2hamy
Copy link
Author

Ok,
now after change app location time to Etc/GMT-3 , the calculation for the next repeated alarm is correct

But if the phone is restarted , it will make a wrong calculation

@islam2hamy
Copy link
Author

🤐,
i have found that the "use app location" option in alarm was activated.
very sorry for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants