Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comment Federation #550

Merged
merged 206 commits into from Dec 22, 2023
Merged

Comment Federation #550

merged 206 commits into from Dec 22, 2023

Conversation

pfefferle
Copy link
Member

@pfefferle pfefferle commented Nov 6, 2023

The initial PR by @mediaformat is already so complex that it is hard to refactor things and make other things work, so I would love to re-start a PR to get through all changes step by step.

Starting with:

  • Permalinks, IDs and Content-Negotiation for Comments
  • Add Inbox of in-reply-to comment to CC (could be that commenter does not follow the blog)

Open Questions:

  • How to handle hashtags in comments? Tags need to be added to a post, at leat the classic ones. WordPress.com has issues with custom taxonomies, so no idea how to support that. Any ideas @mattwiebe or @akirk ?

Proposed changes:

  • Federate comments made on the WordPress blog (limited to WordPress users for now)

Other information:

  • Have you written new tests for your changes, if applicable?

Testing instructions:

  • Go to the comments section of one of your blogs
  • Search for a comment from the Fediverse
  • Write an answer to that comment using the comment form
  • 🎉

@pfefferle
Copy link
Member Author

@mediaformat this is a working first version. I would love to get your feedback on this!

@github-actions github-actions bot added the Docs label Dec 21, 2023
@pfefferle pfefferle merged commit b744dc5 into master Dec 22, 2023
19 checks passed
@pfefferle pfefferle deleted the add/comment-federation branch December 22, 2023 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants