Skip to content
This repository has been archived by the owner on Apr 8, 2020. It is now read-only.

Issue/Bug: 8 or more active alarms makes looping alarm noise and default alarm tone #24

Closed
Tangram11 opened this issue Apr 23, 2016 · 7 comments

Comments

@Tangram11
Copy link

Phone: Samsung S7

Situation to reproduce:

  1. Set alarm tone to something not default (in my case a "click" sounding tone)
  2. Set 8 or 9 events, 1 minute apart, and let them all fire without touching them

Observe:

  • As the alarms fire, the first 7 will sound with the set tone (e.g. the "click").
  • When then 8th tone sounds, it seems to use a default tone - a chime sound.
  • If the screen is active when the 8th or 9th alarm is firing, it gets stuck in a loop and the chime keeps going unless either the screen is turned off or the user can snooze or clear a couple of them.
@quarck
Copy link
Owner

quarck commented Apr 23, 2016

Thank you for reporting, will take a look at this issue in a while

quarck added a commit that referenced this issue May 8, 2016
@quarck
Copy link
Owner

quarck commented May 8, 2016

This should be fixed in 2e269ce, would be available in the build 1.0.23.

I wasn't able to reproduce issue with looping, but I can see what could be the cause, so fix is implemented.

@quarck
Copy link
Owner

quarck commented May 12, 2016

Please reopen if this is still an issue, closing for now

@quarck quarck closed this as completed May 12, 2016
@quarck quarck reopened this May 20, 2016
quarck added a commit that referenced this issue May 20, 2016
@quarck
Copy link
Owner

quarck commented May 20, 2016

Now this should be closed!

@quarck quarck closed this as completed May 20, 2016
@Tangram11
Copy link
Author

I confirm that after the last round of fixes, looping was fixed and the "chime" that was unexpectedly occurring was fixed with it. Great work, thanks!

@quarck
Copy link
Owner

quarck commented May 22, 2016

Thanks for confirming, actually I found another way to reproduce similar issue. When there are more than 8 reminders are snoozed until exactly same time, same issue might happen at snooze time. But this is now also fixed.

@quarck
Copy link
Owner

quarck commented May 22, 2016

New UI would be looking like this:

https://i.imgur.com/8At636L.png

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

No branches or pull requests

2 participants