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

Cannot configure photo backup to save all #1308

Open
leocencetti opened this issue May 16, 2024 · 3 comments
Open

Cannot configure photo backup to save all #1308

leocencetti opened this issue May 16, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@leocencetti
Copy link

Description
When configuring the photo backup to save all photos, the setting is not applied, and the option rolls back to from now on.

Steps to reproduce
Steps to reproduce the behavior:

  1. Go to the photo backup settings
  2. Change the save mode to save all
  3. Click the save button
  4. Reopen the photo backup settings
  5. The save mode is now from now on

Expected behavior
The saving mode should be applied properly. If not compatible with other settings, a warning should be shown.

Screenshots

Before step 2 After step 4
Screenshot_20240516-193836_kDrive Screenshot_20240516-193848_kDrive

Smartphone (please complete the following information):

  • Device: Asus Zenfone 9
  • Android version: Android 14
  • App version: v4.4.8 build 40400801
@leocencetti leocencetti added the bug Something isn't working label May 16, 2024
@leocencetti
Copy link
Author

Looks like this may be a UI-only issue. The synchronization seems to be including all images after all.

@JorisBodin
Copy link
Member

Hello,

This setting only allows you to choose the start date for selecting the photos to be synchronized. Once saved, the worker will scan all photos from this date onwards.
So, strictly speaking, the parameter is not saved, only the date. We could stop displaying this option afterwards to avoid confusion.
So just a problem of understanding

@leocencetti
Copy link
Author

Hey @JorisBodin, thanks for the info.
It would be great if the selection could be reflected in the UI as well, to inform to the user that his configuration has been applied (thus avoiding the confusion).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants