Skip to content

feat: twitch gif messages - #1241

Open
3r01 wants to merge 4 commits into
flex3r:developfrom
3r01:twitch-gif-messages
Open

3r01 wants to merge 4 commits into
flex3r:developfrom
3r01:twitch-gif-messages

Conversation

@3r01

@3r01 3r01 commented Sep 2, 2026

Copy link
Copy Markdown

Adds support for twitch gif messages

@3r01
3r01 marked this pull request as ready for review September 13, 2026 19:18
@flex3r

flex3r commented Sep 16, 2026

Copy link
Copy Markdown
Owner

I'm not too sure how I want to handle Giphy content, since it's kind of third party content that I cannot control really. Of course it's Twitch's job to policy what is allowed to be sent, but Google Play can be very strict in that regard and other Chat Apps got banned for way less. Might introduce this via a developer setting? Not sure.
After fixing conflicts, can you add some explanatory comments, especially how the actual gifs and their image links are constructed? I would have expected that we would need to use some sort of API?

@3r01

3r01 commented Sep 16, 2026

Copy link
Copy Markdown
Author

I had not considered how Google Play might handle it. I assume it does not make any difference from their perspective that the official Twitch app displays the exact same content?

I will add some explanatory comments. No API needed, twitch provides the gif url and its position in the message through the gifs tag and you can load that url directly.

The only reason to manipulate the url is that you can request a different size or format from giphy. For example replacing giphy.gif with 100.webp or 200.webp will return smaller animated webp versions.

# Conflicts:
#	app/src/main/kotlin/com/flxrs/dankchat/ui/chat/messages/PrivMessage.kt
@3r01
3r01 force-pushed the twitch-gif-messages branch from f60a167 to 46a5200 Compare September 16, 2026 13:59
@flex3r

flex3r commented Sep 16, 2026

Copy link
Copy Markdown
Owner

Ah, I hadn't looked at how Twitch provides these gifs. If they are coming through message tags I guess we are fine? I'll test this out this weekend or so. Thanks for all the great PRs btw, coming back from vacation to this was a surprise

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