-
Notifications
You must be signed in to change notification settings - Fork 11
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
checkNewNotifications()
does not work with more energy-efficient HEAD requests in SSO service
#68
Comments
checkNotification()
does not work for SSO servicecheckNotifications()
does not work for SSO service
checkNotifications()
does not work for SSO servicecheckNewNotifications()
does not work for SSO service
Is there any update on this? I'd love to help out if able to. Trying to get my users onboarded and SSO would be nice to have if it works. |
checkNewNotifications()
does not work for SSO servicecheckNewNotifications()
does not work with more energy-efficient HEAD requests in SSO service
@sleibrock Hello. Sorry, probably, my issue title was quite misleading. The issue is about that currently due to limitations(or bugs?) of Nextcloud SSO API I can not do efficient checks for notifications(i.e. HEAD-requests which reduce data usage) |
Thank you, it wasn't clear if this was tied to SSO not functioning, but it appears after a few days of testing that SSO does in fact work for notification polling. |
The following error occurs during call to
checkNotifications()
which is used for traffic-efficient notifications check(see #62)The text was updated successfully, but these errors were encountered: