Skip to content

Conversation

@hannesa2
Copy link
Contributor

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

image

@michaelstingl
Copy link
Contributor

@hannesa2 How does this work? Is this a one-time action that adds older images to the upload queue?

@hannesa2
Copy link
Contributor Author

Exactly.
Imagine, my wife got the new version with #2679 and this works for all pictures from now on. But what's about the past pictures ? Even a reinstall doesn't help.
I haven't this problem, because my phone was broken before #2679 was solved

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

@hannesa2
Copy link
Contributor Author

hannesa2 commented Dec 23, 2019

Btw, before I tried to send/share missed images to ownCloud, but this wasn't possible.
Only chuck size of 200 are allowed by Android (when you have 2500 it's a monkey job and you should not make mistakes), then app was crashing .... so I start to solve it by my own

@jesmrec
Copy link
Collaborator

jesmrec commented Jan 8, 2020

Some comments before including this in the roadmap.

  • Why did you place the option in the bottom of the Settings instead on the Camera uploads section?
  • I installed the branch to play a bit with it (not official QA), and i was not able to set a date till both picture and video are enabled. Shouldn't be two different settings?

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?

@michaelstingl
Copy link
Contributor

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.

@hannesa2
Copy link
Contributor Author

hannesa2 commented Jan 8, 2020

  • Why did you place the option in the bottom of the Settings instead on the Camera uploads section?

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

  • I installed the branch to play a bit with it (not official QA), and i was not able to set a date till both picture and video are enabled. Shouldn't be two different settings?

Of course there is always place for improvements. I put the branch to upstream repo.

@hannesa2
Copy link
Contributor Author

Keep in mind, this very special bug-fix feature of your non-working-sync between 2018-2019 was done only for me.
The longer you wait, the more pointless it is for others.

Should I close it, or is someone interested ?

@jesmrec
Copy link
Collaborator

jesmrec commented Feb 11, 2020

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 Camera Uploads feature. @michaelstingl your comment here means that it is OK?

@michaelstingl
Copy link
Contributor

your comment here means that it is OK?

Yeah, I could imagine such a feature for both Android and iOS app.

@hannesa2
Copy link
Contributor Author

hannesa2 commented Feb 12, 2020

@jesmrec
Thank you about the buzz word quality.

I fixed the broken

  • sync on Android P
  • non-working WiFi-only sync
  • and this PR heals the resulting missing images upload

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

@hannesa2 hannesa2 force-pushed the ChangeSyncInitDate branch from 03798f4 to b314f9c Compare March 5, 2020 05:50
@hannesa2
Copy link
Contributor Author

hannesa2 commented Mar 5, 2020

I solved only conflicts with a rebase to master

@hannesa2 hannesa2 force-pushed the ChangeSyncInitDate branch from 388b648 to d108c14 Compare April 5, 2020 14:18
@hannesa2 hannesa2 changed the title Ability to change sync start date Ability to change sync start date to the past Apr 15, 2020
@hannesa2 hannesa2 closed this Jun 7, 2020
@hannesa2 hannesa2 deleted the ChangeSyncInitDate branch June 22, 2020 10:28
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