-
Notifications
You must be signed in to change notification settings - Fork 80
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
Does this plugin support the "replies" collection? #138
Comments
It is not yet implemented. Do you know a platform that is already consuming the replies-collection? |
Ah, ok - thanks for getting back. There is a new open standard for podcast episode comments that relies on the existing basic Fediverse protocol (ActivityPub, Webfinger, http signatures for auth) for sending and displaying replies to a root comment post associated with that episode. Info site here: https://podcastsocial.org/ For displaying comments, it relies on being able to enumerate comments by parsing the standard For sending comments, it relies on standard s2s federation I was really excited when I saw your plugin, the idea of a podcaster being able to use a WordPress site as the venue for their comments instead of, say, Mastodon, is that WP is already very popular in the podcasting community, and it would presumably provide additional moderation options for these incoming comments.
results in
|
The plugin should accept replies from all big plattforms. The inReplyTo has to be the permalink of the blogpost. |
Is your endpoint accessible? The error is coming from Mod_Security, not from the plugin, so it might be a server configuration issue?!? |
Yep, we're using the article permalink for inReplyTo. But replying doesn't seem to work, even from Mastodon, paging @Lehmancreations for question about security setup on that site |
@johnspurlock There are a few plugins that could maybe be the issue. One being Akismet Anti-Spam and the other is Limit Login Attempts. Let me know when you want to test again and I will disable them |
This is not yet implemented, but it is planned! |
Alright, verified that the 406 response received was due to the shared WP host's custom security filter. They tweaked their secret algorithm and the AP payloads are now allowed through. Phew! One small nitpick remains on incoming comments: your implementation should use the appropriate contentMap lang string (usually only one) as the content text. It currently appears to only recognize the My implementation can include a redundant string property on outgoing requests (Mastodon does this too as a workaround for outgoing), but it's not required, Mastodon/Pleroma use Anyway this is exciting! A fediverse option with fine-grained, pre-emptive moderation is a big deal. While sending works, we won't be able to promote this plugin as a solution, though, until the I understand this might not be a trivial thing to implement, especially if you do it with paged collections for scalability (to support a post with thousands of comments), but this would be much appreciated by several folks interested in pointing people to existing fediverse endpoints on platforms that people are already using. Let me know if I can help in any way, I haven't done PHP in 20 years, but happy to help test on the AP client side. |
I second @johnspurlock's comment Would love to be able to use this plugin in a podcasting setting but would need the replies collection. This would be totally awesome to have cross app comments in podcasting and have the root comment be on our own website were the episode is based. I hope you consider adding the replies collection. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
@mediaformat still eagerly awaiting this - any news on this front? |
@johnspurlock yes, it is planned as part of #142 |
Excellent! Thank you |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
👀 |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This issue is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
keep |
As I learned today the I got interested in implementing this, but got stuck fast, as I stumbled upon the issue whether it is a good practice to leak internal user-ids to an API. |
I would use an API endpoint for this. Something like |
I assume the |
Related: Mobilizon announces every reply with the application actor. Any reasons not to do that (with at least if comments are manually accepted)? ... users should be able to understand the consequences of having followers of the application actor. |
Exactly!
The Short URL is using the post ID, so I wouldn't see an issue here. (Even the User-ID is propagated by the default WordPress API btw.) |
This issue is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
👀 |
This issue is stale because it has been open 120 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
I did not remember that comment, but chose the same design for my draft in #876. This probably confirms that it's a good idea to do it this way. |
@Menrath implemented that, so I will close this for now! please re-open if still an issue! |
Sending an ActivityPub request to
https://www.draky.net/wordpress/2019/11/15/bonjour-tout-le-monde/
which has this plugin installed, returns good looking AP, but without the "replies" collection
https://www.w3.org/TR/activitystreams-vocabulary/#dfn-replies
This makes it impossible to enumerate the comment thread.
I'm wondering if it's implemented?
Thanks
- John
The text was updated successfully, but these errors were encountered: