Skip to content

fix: use thumbnail_uuid for the thumbnail instead of s3_id#132

Merged
costowell merged 1 commit intodevelopfrom
cole-dev
Apr 18, 2026
Merged

fix: use thumbnail_uuid for the thumbnail instead of s3_id#132
costowell merged 1 commit intodevelopfrom
cole-dev

Conversation

@costowell
Copy link
Copy Markdown
Contributor

What

  • Change from using the s3_id to thumbnail_uuid
  • This code is directly copied from the dir thumbnail function

Why

thumbnail_uuid is pretty much always equal to s3_id (save for the file extension which for some reason exists in a column called thumbnail_uuid)

SELECT s3_id, thumbnail_uuid FROM public.files WHERE s3_id != regexp_replace(thumbnail_uuid, '\.[^.]+$', '')

This query yields a single row - the row I modified to get a higher quality video in manually because of our 1Gb upload cap.

Test Plan

  • Noah

Env Vars

N/A

Documentation

Shnope

Checklist

  • Tested all changes locally

Copy link
Copy Markdown
Contributor

@BigSpaceships BigSpaceships left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shippitttttttttttttttttttttttt

@costowell costowell merged commit 6fdf6a1 into develop Apr 18, 2026
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.

2 participants