Skip to content

Conversation

pronskiy
Copy link
Member

@pronskiy pronskiy commented Aug 4, 2025

No description provided.

@pronskiy pronskiy requested review from saundefined and Copilot and removed request for saundefined August 4, 2025 03:48
Copy link
Contributor

github-actions bot commented Aug 4, 2025

@github-actions github-actions bot temporarily deployed to pull request August 4, 2025 03:48 Inactive
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR integrates Reddit comment embedding functionality into blog posts by adding the "redditify" library. The implementation allows blog posts to display Reddit thread comments directly within the post content.

  • Adds redditify library as a dependency and includes its assets in the post template
  • Configures a Reddit thread embed in a specific blog post about PHP's 30th anniversary
  • Sets up the embed with customizable display options for thread depth and controls

Reviewed Changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.

File Description
source/_views/post.html Adds redditify JavaScript and CSS assets to all post pages
source/_posts/2025-06-08-php-30.md Implements Reddit thread embed with configuration options
package.json Adds redditify as a project dependency

@github-actions github-actions bot temporarily deployed to pull request August 4, 2025 09:40 Inactive
@pronskiy
Copy link
Member Author

pronskiy commented Aug 4, 2025

Copy link
Collaborator

@saundefined saundefined left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@pronskiy pronskiy merged commit 1e98b06 into ThePHPF:main Aug 4, 2025
2 checks passed
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