-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Ability to change sync start date to the past #2772
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
Conversation
|
@hannesa2 How does this work? Is this a one-time action that adds older images to the upload queue? |
|
Exactly. That's why this single event action tells the synchronization from which date to start. I know, it's a super special pro feature and nothing for the mainstream, and when you like it, it should be only visible in developer mode. When I write this, I recognize during update to 2.14 a question to the user, which has Wifi sync on, would be useful. "We fixed WiFi synchronization .... do you miss pictures....there you can set a new start date .." |
|
Btw, before I tried to send/share missed images to ownCloud, but this wasn't possible. |
|
Some comments before including this in the roadmap.
Here we have also a product component. Not only Android, also the iOS app implements the "Camera uploads" feature as "enable, and start to upload at this moment". So, will check with PM is this is desired in that way. What do you think? |
iOS app could get such a one-time action as well. No blocker. |
I don't spent any second about this question. And yes, maybe it makes sense to put it to an other place. But for me it was not important, my motivation was to make it work only
Of course there is always place for improvements. I put the branch to upstream repo. |
|
Keep in mind, this very special bug-fix feature of your non-working-sync between 2018-2019 was done only for me. Should I close it, or is someone interested ? |
|
It is OK to keep the PR, every fix minds. But we try to have the product as much correct in all its layers, including the presentation. From my pov, a very good fix or feature must be presented in the correct way. We have many users (and customers)... and giving them an option regarding the camera uploads, that is placed on the bottom of the Settings is not a good way to do, just for giving you an example (recheck this) . I know that the feature/algorithm is where the magic happens, but from a quality pov is not enough. Are you open to make suggested improvements? If anyone in the team has other better ideas, they will be welcome as well Also, for me it is not clear if we are interested in a feature to upload older pictures than the ones after enabling the |
Yeah, I could imagine such a feature for both Android and iOS app. |
|
@jesmrec I fixed the broken
I placed it at the very less in-accesable place I found, because it's a use-only-once option, and should only used when you have problems. But when you have doubt about quality, and think it should be placed on a more prominent place, then I guess this PR has not a glorious future |
03798f4 to
b314f9c
Compare
|
I solved only conflicts with a rebase to master |
b314f9c to
388b648
Compare
388b648 to
d108c14
Compare
As follow up from #2679 I miss tons of pictures on my wife's phone in our ownCloud.
To be able to sync to missing pictures/videos I've to change change sync start date, the only way was to do it in database as a pro-feature
This pull request has hard coded strings, but for me it works. I want to share it with you, but it's up to you how you proceed with: close unmerged, improve or merge