Notifications don't load when many notifications are filtered #3837
Labels
bug
Indicates an unexpected problem or unintended behavior
Comments
mueller-ma
added a commit
to mueller-ma/openhab.android
that referenced
this issue
Dec 2, 2024
Now notifications can be hidden by other notifications which might require that fetching 20 notifications isn't enough to fill one page. Thus increase the page size to 50. Also see openhab#3837
mueller-ma
added a commit
to mueller-ma/openhab.android
that referenced
this issue
Dec 2, 2024
Now notifications can be hidden by other notifications which might require that fetching 20 notifications isn't enough to fill one page. Thus increase the page size to 50. Also see openhab#3837 Signed-off-by: mueller-ma <[email protected]>
mueller-ma
added a commit
that referenced
this issue
Dec 4, 2024
Now notifications can be hidden by other notifications which might require that fetching 20 notifications isn't enough to fill one page. Thus increase the page size to 50. Also see #3837 Signed-off-by: mueller-ma <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Actual behaviour
In the notification fragment due to duplicate reference ids there may be less than 20 (current
PAGE_SIZE) notifications shown and thus the loading indicator is shown after loading the first 20 notifications. It's bound to the recycler view, which triggers fetching the next 20 notifications. When there are too many notifications filtered in this second request, so the loading indicator doesn't disappear, no more requests are made.Expected behaviour
Load until the page is visually full.
The text was updated successfully, but these errors were encountered: