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

Notify and log failures to upload to Nightscout consistently #3875

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Navid200
Copy link
Collaborator

@Navid200 Navid200 commented Jan 30, 2025

When uploading to more than one site, a silent notification is presented and a log is created once a day if there is a failure to upload to one site but not the other.

If some day we add dedicated (per site) queues, all of this will be removed because there will be no need for any of it then.

I am sorry I have added a new setting. Unfortunately, someone may use Nightscout and not care about inconsistent uploads. In that case, having to acknowledge a notification everyday may not be convenient.

Before After
Screenshot_20250130-092153 Screenshot_20250130-080824

Notification
Screenshot_20250130-080725

Log
Screenshot_20250130-080638

Translations
The notification and its title will be translated.
The log will not be translated.

Tests
This has been tested on Motorola Android 11 and Motorola Android 8.
There is no notification if all uploads fail (no WiFi).
There is notification and log when there is a failure to upload to one site but not the other. By default, the notification is enabled.
Disabling the notification will still generate a log but no notification.
To cause failure to upload to only one site, the Google Cloud Nightscout site was temporarily stopped.

@Navid200 Navid200 changed the title Notify and log failure to upload to Nightscout consistently Notify and log failures to upload to Nightscout consistently Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant