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

[TOP] Enhance sync folders #285

Open
5 of 6 tasks
tobiasKaminsky opened this issue Sep 21, 2016 · 168 comments
Open
5 of 6 tasks

[TOP] Enhance sync folders #285

tobiasKaminsky opened this issue Sep 21, 2016 · 168 comments

Comments

@tobiasKaminsky
Copy link
Member

tobiasKaminsky commented Sep 21, 2016

As discussed on conf this is a collaboration between @AndyScherzinger and me.
Purpose:

  • Increment 1 - instant upload - Auto Upload (Android 7+) #358 - release 1.4.0 ✔️

    • replace current instant upload
    • full support for Android 6,7
  • Increment 2 - enhancement auto upload - Add further media types to auto upload #797 - release 2.0.0 ✔️

    • support video folders
    • allow any kind of folder pair (custom folders)
    • show jobs in upload pane
      • track jobs
      • get job result to update upload pane
      • if job fails: allow user to restart
    • automatic retry
    • upload updated files
  • Increment 3 - upload existing files ✔️

    • add "upload/sync all existing images/videos/files" as an option to the auto upload folder three-dot menu.
  • Increment 4 - folder upload ✔️

    • allow folder hierarchies
    • smart UI to semi-automatically detect and notify about potential new media folders the user can add/active for instant upload New media detection #2406
  • Increment 5 - sync one way

    • sync - one way
    • free up space
  • Increment 6 - sync two way

@aparcar
Copy link

aparcar commented Sep 23, 2016

Does this include a constant sync of a todo.txt file on change locally and periodically checked on server?

@tobiasKaminsky
Copy link
Member Author

@aparcar currently this will only replace instant upload.
But this approach is designed to also handle a real two way sync and a constant sync of files.

@AndyScherzinger
Copy link
Member

This issue/implementation is to be considered the top prio since we now already have > 1,300 Android 7 installations according to the play store ❗

@AndyScherzinger AndyScherzinger modified the milestones: Nextcloud App 1.5.0, Nextcloud App 1.4.0 Oct 13, 2016
@tobiasKaminsky
Copy link
Member Author

Camera should be top of the list as this is the most wanted "folder", I guess.
At least I was searching a while where it is ;-)

@tobiasKaminsky
Copy link
Member Author

Uhh, and caching is not working properly, right?

@tobiasKaminsky
Copy link
Member Author

I know it is kind of reduntant, but I think that it should be possible to enable/disable sync also in the setting of a folder (and not only in the folder sync view).
[folder sync looks fantastic! 👍 🎉 ]

@AndyScherzinger
Copy link
Member

Yeah, we can put camera on top, need to think how to put this into the comparator. Doable :)
As for the caching, don't know actually. The queries need to be tweaked also since they need to order by date.

@AndyScherzinger
Copy link
Member

And yes, the enable/disable can also be put into the settings. UI is close to feature complete. Then I will publish some screenshots for the discussion with the designers.

@AndyScherzinger
Copy link
Member

Camera folder at the top (of the inactive): DONE

@AndyScherzinger
Copy link
Member

I am with @Ablu, the feature has always worked for me on my OP3T which runs on Lineage OS nightlies.

@Cyber1000
Copy link

For me (Honor 8 with Emui 8/Android 8), it works without problems too, 100% for the last 6 months with mainly photos/videos. But I upload only on wifi and if connected to power and have disabled the aggressive power saving of honor 8 for this app.

@lemmy04
Copy link

lemmy04 commented Oct 25, 2019

I'm sorry to be negative about it, but on all my devices "auto" upload means that for about 50% of all the pictures I take I have to manually open the nextcloud app, go to the "uploads" tab, and swipe down to refresh, to get the app to actually see the new pictures and realize that there are uploads to be done.

And there seems to be no logic behind then and why it happens...

The only connecting factor seems to be the fact that all my devices store their pictures on SD card, I could reactivate my old S6 that does not have one and see what happens on that one.

@Janhouse
Copy link

Janhouse commented Oct 25, 2019

I noticed that on new Android devices if app doesn't have the "autostart" permission (which is not given by default), and if Nextcloud is off while the photo is taken, it never figures out that it has to upload something unless you open the app and try to refresh something. (Tested on Xiaomi device)
So you have to give every sleep related permission to the app manually and then it seems to work more reliably.

Meanwhile on iOS the Nextcloud app uses some sort of geofence feature that works around the lack of "autostart" on iOS. Basically when the location changes, it is allowed to wake up and do some work for limited time.

I guess Android app has to be updated to be more in line with newer Android versions.

@lemmy04
Copy link

lemmy04 commented Oct 25, 2019

I checked: on my most current device, a Samsung Tab S5e running Android 9, the app does have the autostart permission.
What I don't have anymore on that device is the option of "locking" the running app so that "close all apps" leaves it alone, which sort of seems to be a working workaround on my phone...

@rugk
Copy link

rugk commented Oct 25, 2019

I guess you issues generally are the heavy optimization of Android and especially some vendors trying to prevent high battery usage and thus stop/kill many, many apps. See https://dontkillmyapp.com/ for potential solutions or similar.

@jospoortvliet
Copy link
Member

I think @rugk is right. This auto-upload feature works fine for the vast majority of phones, and isn't the thing we are discussing here - this topic was about full folder synchronization.

Sadly some phone vendors do crazy stuff that breaks auto upload, and it is near impossible to work around every weird thing every single vendor does PLUS keep up with all new weird things they do on new versions of their phones. If your phone is one where things don't work well and you want it fixed, complaining won't do much. Determine what is wrong and provide information - because we don't have your phone so we don't know what is wrong and how to fix it... At the very least, create an issue with as much info as possible - like, in what situations DOES it sync (when the app is running? When you disabled all powermanagement? etc), when it doesn't, and of course exactly what phone and software you use. Maybe a bit of googling can tell you or somebody else how to fix it and that fix might possibly be applied to our app.

@Janhouse
Copy link

@jospoortvliet, no, it does not really work fine on Android. It uploads only files created after auto upload was enabled. This means that if I have phone with lots of photos and I install Nextcloud, none of those images will be automatically uploaded.
iOS app doesn't have this problem.

@AndyScherzinger
Copy link
Member

@Janhouse this is known and has never been implemented and thus doesn't work on any device running Android.
So this isn't even a Sync discussion while your point then simply is implementing existing pictures when auto upload is being activated.

What I don't know is how iOS client is handling this (technically speaking). Maybe somebody can shed some light on this?

@jospoortvliet
Copy link
Member

@Janhouse yeah, I personally would also like that feature to be implemented, no doubt that'd be cool.

A full 'sync' that is two-way is very hard, but a one-time upload-all for a folder would be nice. Or even a dumb "upload unless file exists" or even just bloody overwrite them... would all be fine for me.

@AndyScherzinger
Copy link
Member

A full 'sync' that is two-way is very hard, but a one-time upload-all for a folder would be nice. Or even a dumb "upload unless file exists" or even just bloody overwrite them... would all be fine for me.

As for "upload existing", there is #2873 while is a starting point but nobody has been able to get the people involved to an common understanding and agreement hwo this should wor exactly in the first place thus this feature is simply "pending".

@MostFrabjous
Copy link

What I don't know is how iOS client is handling this (technically speaking). Maybe somebody can shed some light on this?

I also am really interested!

@Janhouse
Copy link

Janhouse commented Nov 3, 2019

Probably keeps list of files locally and marks which files don't have to be uploaded again.
Android app on the other hand does this weird Cursor thing.

@jospoortvliet
Copy link
Member

let me link to the successor PR: #4788

@mjflower
Copy link

I can't believe this is still an issue after all this time. I recently installed the app on the kids (have to have an apple because the school mandates it) devices and it uploaded all previously taken images without issue. Why can the android apps have this option, I don't get it.

@jospoortvliet
Copy link
Member

I can't believe this is still an issue after all this time. I recently installed the app on the kids (have to have an apple because the school mandates it) devices and it uploaded all previously taken images without issue. Why can the android apps have this option, I don't get it.

Oh, it isn't so hard to understand. Why did you not write this feature?

No time? I can imagine. Happens to be true for everyone here, otherwise it had be done by somebody. I personally would love this feature, but I don't have time to learn programming and then do it. Hope that helps explain why it isn't done.

If you really want this done, you can do it, of course, your kids and everyone else will be very grateful if you did!

Until then, I'll close this discussion as it isn't going anywhere.

@nextcloud nextcloud locked as too heated and limited conversation to collaborators Feb 26, 2020
@jospoortvliet
Copy link
Member

@tobiasKaminsky pretty much all of this is done, correct? Perhaps we can then close this issue?

@AndyScherzinger
Copy link
Member

@jospoortvliet increments 5 and 6 are open

@joshtrichards joshtrichards added the feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific label Oct 9, 2023
@jancborchardt jancborchardt moved this to 🧭 Planning evaluation / ideas in 🖍 Design team Dec 21, 2023
@joshtrichards joshtrichards removed the feature: transfers [WIP] Any uploading / downloading matters that aren't auto upload or sync specific label Oct 3, 2024
@jancborchardt
Copy link
Member

@tobiasKaminsky meanwhile, only one item is open, and it has its own issue at #19 – so I would suggest closing this overview in favor of the single issue?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests