Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add backup to Google Drive #4481

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

Open
Angus-repo wants to merge 5 commits into nightscout:dev
base: dev
Choose a base branch
Loading
from Angus-repo:google-drive-backup

Conversation

@Angus-repo
Copy link
Contributor

@Angus-repo Angus-repo commented Jan 10, 2026
edited
Loading

This PR implements the functionality requested in #1208 and #3296.
Thanks to the preparatory work completed by @vanelsberg in PR #3487, this PR is able to support automatic export of settings.

This feature allows users to select a cloud directory from the maintenance page. The original AAPS directory has been renamed to Local Directory; for first-time use, users are still required to specify the local path.
Users can independently decide whether logs, settings, and user entries should be stored in the cloud. The settings feature also supports importing settings from the cloud.

Due to slower cloud import performance, only the latest 5 settings entries are displayed at a time. If needed, users can manually load the next 5 entries.

Settings export can be combined with Automation: once the pump is enabled, settings are automatically exported to the selected cloud directory.
(Note: users are still required to manually enter their password once every four weeks.)

The main functionality is complete. Please help test this and provide feedback. Thank you!

圖片 圖片 圖片 圖片 圖片

vanelsberg, Koeppchen, olorinmaia, xitation, and asavageiv reacted with heart emoji
Copy link
Contributor

MilosKozak commented Jan 10, 2026
edited
Loading

are you sure it's a good idea to store log out of local storage. it's changing a lot

Copy link
Contributor Author

The logs are still stored locally (in EXT_FILES_DIR used by logback). Only when exporting will the export location be determined based on the user’s selection. To avoid any misunderstanding, I adjusted the wording and have updated the images.

vanelsberg and olorinmaia reacted with thumbs up emoji

Copy link
Contributor

vanelsberg commented Jan 11, 2026
edited
Loading

Great to see to his PR!

PR merged with dev, phone updated without issues, Did not encounter any issues on functionality.
Well done @Angus-repo!

Only feedback I have:
on initial Google authentication, "AAPS-CI" was mentioned. Could confuse user. Is this intentional?

Will run this on my main phone the coming days/weeks for more "field" testing.
TODO: test new installation, restore settings.

Angus-repo reacted with thumbs up emoji

Copy link
Contributor Author

@vanelsberg see AAPS-CI because it was created using the same website for the Google OAuth2 application. I will apply for an update to change the name and the icon.

vanelsberg reacted with heart emoji

Copy link
Contributor

olorinmaia commented Jan 11, 2026
edited
Loading

Great work @Angus-repo ! I will also test when I got time :)

Angus-repo reacted with thumbs up emoji

Copy link
Contributor Author

Optimize cloud directory prompt message(Sync updated first-page display)

圖片 圖片
vanelsberg reacted with thumbs up emoji

Copy link
Contributor

vanelsberg commented Jan 12, 2026
edited
Loading

PR does great and merges OK with current dev. No issues until know. Top!

@Angus-repo: Feature request?
In general, I have settings stored locally and use cloud as a backup. As is now we have local or cloud.
Unless this would need a lot a code changes, could we have both?

Der-Schubi and xitation reacted with thumbs up emoji

Copy link
Contributor Author

Angus-repo commented Jan 13, 2026
edited
Loading

@vanelsberg

Updated. Please try again.(Sync updated first-page display)

圖片 圖片
vanelsberg and xitation reacted with thumbs up emoji

Copy link

Copy link
Contributor

Updated. Please try again.(Sync updated first-page display)

Yes :-) Thanks! This works great.
Exporting to cloud or local works without issues.
Selecting both "local" and "cloud" for preferences also works as expected.

FYI: I'm running latest dev (FullRelease, daily builds) with your PR merged on my daily driver phone.
Did not encounter any issues!

Angus-repo reacted with thumbs up emoji

Copy link
Contributor

vanelsberg commented Jan 18, 2026
edited
Loading

This PR is still going strong for me. Imho it's complete and done? Ready for merging with dev.

@Angus-repo Angus-repo marked this pull request as ready for review January 20, 2026 03:45
Copy link
Contributor Author

@MilosKozak

I’ve changed this PR to Review status. If there are significant conflicts with the upcoming 4.0.0 version or if architectural adjustments are required, please tag me and I’ll make the necessary changes.

Note: The name and logo on the Google sign-in page are still under Google’s review. Once approved, they will take effect automatically without any code changes, so the sign-in page message can be ignored for now.

vanelsberg reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /