This repository has been archived by the owner on Oct 16, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 0
Remove image and page number from DB #76
Labels
Comments
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Just commented them out in the previous commits. Checked, it is now working so deleted it. Tests will now obviously fail though (photo model tests etc that is)
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
This is only used when uploading analog scans, will have to find a fix to the now arising analog physical archival problem in the future.
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
…omponent #76 We now need to find a new way to upload scanned images correctly
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Mikkel94
pushed a commit
that referenced
this issue
Jan 29, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
This needs to be done so we can have parallell upload of images. As of right now this is impossible because per image uploaded we are GETing the next page/image number from the backend, so when 2 (or more) people are trying to upload photos at the same time it will crash for one (or more [or all]) of them. Will do a redesign of the DB for this purpose.
We have to keep page and image number on the analog photos for physical archive purposes. I'm thinking we will do this through just a comment instead. That might be the best.
The text was updated successfully, but these errors were encountered: