gocast icon indicating copy to clipboard operation
gocast copied to clipboard

Published 20 hours ago • verified publisher icon TUM-Dev

Thumbnails versions mixed up

Open alexanderstephan opened this issue 2 years ago • 3 comments

Describe the bug Sometimes the versions of the thumbnails get mixed up. This is somehow related to the latest changes, I think it stopped working after the latest deployment.

Potential issue: https://github.com/joschahenningsen/TUM-Live/blob/ea2913f09ecb143961ec505b337b8959f3a8db0f/model/file.go#L15

To Reproduce Steps to reproduce the behavior:

  1. Go to https://live.rbg.tum.de/w/I2DL/12354/CAM
  2. Hover over timeline
  3. Observe thumbnails for presentation instead of camera

Expected behavior The thumbnail version should match the stream version.

Additional context The issue can somehow not be reproduced on the testserver. There all version seem to match, see https://test.live.mm.rbg.tum.de/w/airplay/498/CAM

alexanderstephan avatar Jun 27 '22 22:06 alexanderstephan

todo: move FILETYPE_IMAGE_JPG to bottom, delete section previews

joschahenningsen avatar Jun 28 '22 08:06 joschahenningsen

Another option would be to just migrate the existing thumbnail types to the new logic, so +1 in the database for all thumb types (until yesterday I guess).

alexanderstephan avatar Jun 28 '22 09:06 alexanderstephan

That would be even cleaner I suppose.

joschahenningsen avatar Jun 28 '22 12:06 joschahenningsen

This can also be tracked in #536 I think

joschahenningsen avatar Oct 26 '22 08:10 joschahenningsen