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

Notifications: dicuss polling #1561

Closed
tobiasKaminsky opened this issue Sep 20, 2017 · 7 comments
Closed

Notifications: dicuss polling #1561

tobiasKaminsky opened this issue Sep 20, 2017 · 7 comments
Labels
enhancement feature: activity and notification Server activity and notifications hotspot: push notifications Push notification distribution (and, in theory, pull) needs info Waiting for info from user(s). Issues with this label will auto-stale.

Comments

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Sep 20, 2017

If push notifications are not working, e.g. on Fdroid or temporary problems of registration, do we want to poll?

@AndyScherzinger #1560 (comment)

[...] would recommend to have a 10-15 minutes interval to not drain the battery. As a second step we could then of course go for a "smarter" polling looking at the frequency of notifications coming in.

@mario #1560 (comment)

You have to use AlarmManager for F-droid as that is the only thing that currently guarantees the device will wake up. The interval can be decided I guess, but in order to know which notifications to show we'll have to store notifications in the DB and show only ones we haven't shown etc etc.

@tobiasKaminsky tobiasKaminsky added this to the Nextcloud App 2.1.0 milestone Sep 20, 2017
@srkunze
Copy link
Member

srkunze commented Sep 21, 2017

I would suggest using an increasing interval in case of silence. Had good experience with either linear or exponential growth.

Just wonder if there isn't a standard solution for this in Android (push fallback or something).

@mario
Copy link
Contributor

mario commented Sep 21, 2017 via email

@srkunze
Copy link
Member

srkunze commented Sep 21, 2017

No. Fallback is manual polling :)

Manual is meh :D all Android apps have to implement it themselves. What a waste...

@mario
Copy link
Contributor

mario commented Sep 21, 2017 via email

@srkunze
Copy link
Member

srkunze commented Sep 21, 2017

That's only if you don't want to publish on Play Store. All Play Store apps can have push without an issue.

Ah, okay. Didn't know. 👍

@AndyScherzinger AndyScherzinger added enhancement needs info Waiting for info from user(s). Issues with this label will auto-stale. labels Sep 25, 2017
@tobiasKaminsky
Copy link
Member Author

Fallback is manual polling :)

This means that an user has to enter the app and opens "Notifications". But with polling, even if it is only once an hour, the user will get a delayed notification without manual interaction...

@tobiasKaminsky tobiasKaminsky removed this from the Nextcloud App 2.1.0 milestone Jan 2, 2018
@joshtrichards joshtrichards added the feature: activity and notification Server activity and notifications label Oct 9, 2023
@joshtrichards joshtrichards added the hotspot: push notifications Push notification distribution (and, in theory, pull) label Dec 8, 2023
@joshtrichards
Copy link
Member

Let's consolidate with the more recently active #7046

@joshtrichards joshtrichards closed this as not planned Won't fix, can't repro, duplicate, stale Dec 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature: activity and notification Server activity and notifications hotspot: push notifications Push notification distribution (and, in theory, pull) needs info Waiting for info from user(s). Issues with this label will auto-stale.
Projects
None yet
Development

No branches or pull requests

5 participants