OSS-DocumentScanner icon indicating copy to clipboard operation
OSS-DocumentScanner copied to clipboard

[BUG] iOS application does not work with WebDAV set up from the android app

Open DhruvaSambrani opened this issue 1 year ago • 6 comments

Version

1.5.0

What platform are you using?

iOS

OS Version

Latest

What happened?

Title.

Exact error: Unique constraint failed document.Id

Relevant log output

No response

Code of Conduct

  • [X] I agree to follow this project's Code of Conduct

DhruvaSambrani avatar May 14 '24 20:05 DhruvaSambrani

I think the error is because the FoxyDroid version is 1.8.0/72, but the IOS store version is older. Is it possible to update the IOS app? You may also want to use TestFlight incase the IOS store has some update rate limiting

DhruvaSambrani avatar May 15 '24 07:05 DhruvaSambrani

@DhruvaSambrani i got a bug report from your iOS side. It is weird it feels like your database is in a wrong state. Can you try to clear data on iOS ? (might have to uninstall then install again) And then try again. It could work, though it wont explain what happened. Did you do anything specific before on the iOS app? So you understand the sync service see a document which needs to be downloaded but the id of the document is already existing in your database (though no local file). It could be a bug where you deleted the local document but the it was not correctly removed from database

farfromrefug avatar May 15 '24 13:05 farfromrefug

Reinstalled and tried to set up the webdav.

This time, the first error is table Page has no column named brightness, and then when I try it again, it gives me the unique constraint failure.

The failed import on the first sync leaves the db in a bad state, it seems.

DhruvaSambrani avatar May 15 '24 15:05 DhruvaSambrani

@DhruvaSambrani i see what s happening now. And you were right it could be due to the incompatible versions. The android version does support that new brightness column while the ios version does not. Hopefully new iOS version coming real soon. Sorry about the trouble

Still an issue though that such an error table Page has no column named brightness should not put your db in a bad state. Will look at it

farfromrefug avatar May 15 '24 16:05 farfromrefug

Also, it might be useful to add some checks so that backward compatibility is somewhat maintained.

DhruvaSambrani avatar May 16 '24 11:05 DhruvaSambrani

@DhruvaSambrani it is all implemented and new iOS version is waiting for review

farfromrefug avatar May 16 '24 14:05 farfromrefug