Skip to content

feat: proxy and embed giphy gifs in comments - #561

Open
ksharizard wants to merge 1 commit into
redlib-org:mainfrom
ksharizard:giphy-embeds
Open

ksharizard wants to merge 1 commit into
redlib-org:mainfrom
ksharizard:giphy-embeds

Conversation

@ksharizard

Copy link
Copy Markdown

Closes #547

Before, the link would be shown without any embedding. To see the gif, you would need to open the gif link. Now, the gif is proxied through the instance and embedded to the user. Clicking on the gif opens the gif through the proxied link.

Example: r/cats/comments/1ugv599/wake_up_complaint_and_goes_to_sleep_again/ou30fs7/?context=3#ou30fs7

Before:
image

After (looping gif):
image

@exilonium exilonium left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM
works !!!

@mzhboy

mzhboy commented Jul 22, 2026

Copy link
Copy Markdown

yes, it works! 👍,👍

Sparronator9999 added a commit to Sparronator9999/redlib that referenced this pull request Aug 6, 2026
feat: proxy and embed giphy gifs in comments
Sparronator9999 added a commit to Sparronator9999/redlib that referenced this pull request Aug 6, 2026
feat: proxy and embed giphy gifs in comments
LucifersCircle added a commit to LucifersCircle/redlib that referenced this pull request Sep 15, 2026
* Add selected upstream quality-of-life improvements

Integrates redlib-org/redlib PRs redlib-org#194, redlib-org#291, redlib-org#394, redlib-org#506, redlib-org#520, redlib-org#521, redlib-org#524, redlib-org#539, redlib-org#542, redlib-org#546, redlib-org#560, and redlib-org#561.

* Format existing RSS helpers

Normalizes the upstream RSS code already present on main so the repository-wide rustfmt PR check passes.

* Satisfy current Clippy checks

Applies behavior-preserving fixes for warnings promoted to errors by the pull-request workflow, including the imported CMAF URL formatter and pre-existing RSS/duplicates code.
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.

✨ Feature parity: Embed giphy links

3 participants