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

The SAS administration (and user) interface #94

Open
Juknum opened this issue Mar 26, 2020 · 0 comments
Open

The SAS administration (and user) interface #94

Juknum opened this issue Mar 26, 2020 · 0 comments
Labels

Comments

@Juknum
Copy link
Member

Juknum commented Mar 26, 2020

In GitLab by @tleb on Mar 26, 2020, 16:50

The SAS administration interface is broken. I had a meeting with someone from the "Club Reflex", here are the problems they have:

  1. No way to delete an entire album.
  2. Deleting more than ~3 pictures is broken.
  3. The pagination is too small. They suggested fake pictures at the start and at the end of each page to go to the next/previous one (if applicable). It seems like a good way to show that there are more pictures (people don't seem to notice the pagination).
  4. How are pictures sorted? Seems like it's based on the upload order, which isn't ideal. Solutions would be datetime-based or filename-based (their cameras create filenames which can be sorted so that the result is the same as datetime sorted, but datetime-based would be better for when they have more than one camera).
  5. The upload process is broken. They can't upload a full album (a few gigabytes) without an error (which one?) and need to restart the whole process. However, duplicates are not handled properly and so they need to delete duplicates once everything is uploaded.
  6. No way to hide albums so once the upload is started, it's visible to everyone.
  7. Moderation requests have no message associated to them.

We should start by trying to find (replicate and/or find on sentry) the errors that occur during deletion and upload. We should be able to try it directly on the prod server as it's pretty idle ATM (maybe add the visibility parameter first). :)

Random thoughts:

  1. Deletion should maybe be made async with requests to an API, the issue is probably a timeout as deleting many images might be too expensive. However, it seems weird that it bugs at ~3 images, I would have expected deleting 3 images taking only a few hundred milliseconds and therefore not being a problem.
  2. One quick way to deal with the upload process issue would be to avoid re-uploading the same pictures (based on a hash or the filename) on the front-end. Currently, they have to restart the whole process and therefore have as much chances for it to fail. If the front-end ignored already uploaded images, they would just have to restart the process with all images selected and it would only what is not yet uploaded. We need to deal properly with images which did not finish uploading!
  3. Prettifying the SAS would be a rather easy step: remove the frames, borders and box-shadow (or reduce it largely).

Opinions?

@Juknum Juknum added this to Sith v3 Dec 12, 2022
@Juknum Juknum moved this to A Vérifier in Sith v3 Dec 12, 2022
@TheoDurr TheoDurr moved this from A Vérifier to Todo in Sith v3 Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Todo
Development

No branches or pull requests

1 participant