-
-
Notifications
You must be signed in to change notification settings - Fork 379
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
Tusky notification-refactor branch got confused about filters, then crashed #4433
Comments
Yesterday upgraded to a release build of 05c7e7b (develop branch head yesterday). This morning I again reproduced the "boosts and favorites are turned off, but still appearing in the notification column". It may be this is 100% reproducible and you just need to turn off boosts and favs in the filters, make a high-visibility post that will generate diverse notifications, turn off the phone and go to bed. The crash did not reproduce. Screenshot: |
This does four things - set `enablePlaceholders = false` on `PagingConfig`s to avoid Paging Data that contains null placeholders, we don't want them (everywhere, not just in notifications) - make sure NotificationsPagingAdapter does not crash when it encounters a null placeholder - makes sure the notifications refresh correctly when the filters change - the filters are now also respected when loading a gap closes #4433
Detailed description
This morning I woke up and scrolled through my Tusky notifications. Because of Arguments, I had an unusually large number of notifications. I was finding it hard to see the replies among the boosts, so I tapped "filter" to hide the boosts and favs. To my surprise, "favs" and "boosts" were already disabled in the filter settings, although favs and boosts were definitely visible. Experimentally, I tapped "Apply" without making any changes. The app crashed ("this app has stopped"). Afterward I found this stack trace in my logcat:
tusky-crash-snip-2024-05-09.txt
On rebooting, the inconsistent state (boosts and favs visible, but unchecked in filter dropdown) persisted. "Refreshing" (IE dragging down from top) did not fix the visibility states as it sometimes does. Clicking "Apply" did not fix the visibility states (although also, it did not cause another crash). Checking "boosts" and "Favs" and hitting apply, then unchecking and hitting apply again, did fix the visibility states.
Steps to reproduce the problem
No known reproduction
Debug information
5ba10f2d2067
(non-final commit from notification refactor branch), self-built, greenDebugThe text was updated successfully, but these errors were encountered: