Skip to content

Conversation

@Angus-repo
Copy link
Contributor

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

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!

圖片 圖片 圖片 圖片 圖片

@MilosKozak
Copy link
Contributor

MilosKozak commented Jan 10, 2026

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

@Angus-repo
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
Copy link
Contributor

vanelsberg commented Jan 11, 2026

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
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.

@olorinmaia
Copy link
Contributor

olorinmaia commented Jan 11, 2026

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

@Angus-repo
Copy link
Contributor Author

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

圖片 圖片

@vanelsberg
Copy link
Contributor

vanelsberg commented Jan 12, 2026

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?

@Angus-repo
Copy link
Contributor Author

Angus-repo commented Jan 13, 2026

@vanelsberg

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

圖片 圖片

@sonarqubecloud
Copy link

@vanelsberg
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!

@vanelsberg
Copy link
Contributor

vanelsberg commented Jan 18, 2026

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
@Angus-repo
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.

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.

4 participants