single frame gifs get marked as animated though converted to static by Discord

[ioistired]

This is a one-frame gif:

Discord detects this, converts to a png, and uses one static slot in the guild. However, mime_type_for_image returns image/gif, so animated is set to True in the database. Since we need the image type in order to pick a guild to upload to, probably the best solution is to detect single-frame gifs client side before uploading. Or we could just convert them to fake two frame gifs as Emote Manager now does.

[ioistired]

Note that this actually causes emote display to break:

[StarrFox]

I think we should convert them to two frame gifs as this is probably what the person adding the emoji intended to happen

[ioistired]

@StarrFox why not convert them to pngs instead?

[ioistired]

That way the image itself would be usable as an emoji for non-nitro folks too

[StarrFox]

they can just change the file type if they wanted to add it to their own server