[Bug] Notifications. (android issue???)

[boydkelly]

When I have notifications enabled, i get notified over and over and over and over and over and over and over and over...

If not 5 seconds then 5 min after I dismiss.

Is this an android problem? I wouldn't mind getting a notification once (or heck twice) a day. But 5 seconds after I dismiss the anki notification it comes back. Why is this happeing?

[mikehardy]

Mostly because our notifications sub-system needs a lot of work, and it hasn't been a priority. I feel your pain but also just swipe away the notifications as I haven't had time personally to implement a fix 🤷

[david-allison]

It's to do with the widget, Best to disable the notifications in the app/OS settings. Mirroring mike, we need someone who cares about the notifications to spend a few days fixing them, but it hasn't hit my priority yet as I don't use them.

[moqi-Git]

I want to try fixing this but I can't reproduce on my device. Which kind of notification bothered you?

[boydkelly]

Hi,
I was probably in a grumpy mood the morning I posted that, as this is nothing seriously affecting Anki. But I have under settings general notifications. Notify when more than 75 cards due. Probably because I have 4 decks and several filtered decks, I get 3 or 4 notifications when I look at my phone in the morning. But the issue is if I dismiss the notifications, because I just can't study, they will come back again, sometimes within seconds, and I have to keep dismissing them. Hey I was actually finding that kind of funny at first that Anki was so 'determined' to get me on the right track! But then after a while with constant notifications all day it becomes wearysome. So they are off now. No biggie. If you can make notifications come on only once in a 24 hr period that would be better IMO.

[mikehardy]

I reproduce the same behavior, and think I even wrote the code that did it! Because, awesome 😅 - hey, more notifications is better than 0? But in this case only slightly

[2tanayk]

Mostly because our notifications sub-system needs a lot of work, and it hasn't been a priority. I feel your pain but also just swipe away the notifications as I haven't had time personally to implement a fix 🤷

will do the work on the notifications thing, let me reproduce the issue and I will get started then 👍

[2tanayk]

Thanks for assigning me this issue @david-allison-1 sir!
It would really be great if you provide me with a bit of guidance for this issue as I do not know much about the notifications sub-system (although I will search and start studying the code as soon as possible).
Just brief me about any key aspects of how it works right now, Thanks in advance!

[mikehardy]

#4944

[2tanayk]

@mikehardy sir first of all thanks for linking that discussion to me,but that discussion is 2 years old and as someone who is new to this project (I might be asking too much) I would really like to get a gist of what exactly needs to be done, like in points, would really help.
That being said I've already started studying the code and using the existing resources to understand the current system in depth

[david-allison]

https://developer.android.com/guide/topics/ui/notifiers/notifications

https://github.com/ankidroid/Anki-Android/blob/master/AnkiDroid/src/main/java/com/ichi2/anki/services/NotificationService.java

https://github.com/ankidroid/Anki-Android/blob/master/AnkiDroid/src/main/java/com/ichi2/anki/NotificationChannels.java

and related history.

Nothing's really changed in the last 2 years, just needs someone to form a plan, explain how we currently use notifications, and get it working.

Do note that notification channels are immutable from the program once created - you can't update values, only the user can with their notification settings.

This means things such as the vibrate pattern are effectively "write-once", as we can't change them, and Android doesn't provide an option for the user to change them

[Arthur-Milchior]

Do you get one notification by deck or do you get the notification for the same deck everytime? Because one problem is that there are notifications for each top level deck potentially. I don't like it but was told to let it this way.
I can imagine that android limit notification to a small number and queue the next notification until the first one are seen.

[mikehardy]

Android I think has a limit of 50 notifications at once, and some phones (Samsung I think?) limit to 500 pending intents (think "alarms") set at once.

I think the first thing to solve is that if notifications are swiped away for the day, they actually stay away until the next day

[2tanayk]

Working on this

[haran2248]

Can I try taking up this issue as well?If it is ok with @2tanayk

[2tanayk]

Can I try taking up this issue as well?If it is ok with @2tanayk

I don't think two people should work at the same time

[prateek-singh-3212]

Hi @2tanayk,
If you have any need, I would love to help you with this issue.

[2tanayk]

Hi @2tanayk,
If you have any need, I would love to help you with this issue.

Sure thanks!

[2tanayk]

I am working on it,was a bit busy(because of college 😬),will get it done soon :)

[david-allison]

@2tanayk Life comes first, take your time, grades are important.

[2tanayk]

if some1 else wants they can take this issue as I have not been able to spend ample time at this ,really sorry for any inconvenience caused :(,but I'll definitely attend to it if nobody does

[david-allison]

Don't worry! Life comes first. Give me a ping when you have time

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[eskoONE]

Can I try taking up this issue as well?If it is ok with @2tanayk

Hi @2tanayk,
If you have any need, I would love to help you with this issue.

Do you guys want to have another look at this issue? The guy that wanted to work on this didn't have the time to do so and left it open for others. Since nobody seems to be working on this anymore, I wanted to make sure you guys have seen this, since you showed interest on working on this issue.

I think ankidroid is in a dire need of a notification system overhaul, since its practically not very useful and difficult to use (perhaps not really difficult, more like confusing. Its weird to have a widget on the home screen to have notifications to work at all).

Would appreciate if one of you guys or both of you could figure out a better way of implementing notifications for ankidroid.

Thanks!

[eskoONE]

I meant to ping @haran2248 and @prateek-singh-3212, please read the comment above.

Sry about the double post.

[prateek-singh-3212]

@eskoONE Thanks for reminding me about this issue. Currently their are 3 issues of ankidroid in my todo. But I will take this one because I want to brush up my notification concepts.

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[eskoONE]

@prateek-singh-3212 any news on this?

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[prateek-singh-3212]

Working on it!!

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically

[github-actions]

Hello 👋, this issue has been opened for more than 2 months with no activity on it. If the issue is still here, please keep in mind that we need community support and help to fix it! Just comment something like still searching for solutions and if you found one, please open a pull request! You have 7 days until this gets closed automatically