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
Store Webmention Responses in Post Meta #23
Comments
The idea is good, but I am not sure if this should be part of the core plugin, because it is very brid.gy specific. The spec doesn't specify the response at all, so I think it would be cool to code a bird.gy plugin that hooks into the "send_webmention" action (if added a filter today bf62f8f#diff-a206f8eddc4eec457824d08af6ecd005R319) what do you think @snarfed? |
This was the debate I was trying to figure out. Should the response be
|
actually, the webmention spec strongly encourages a response body: The response body SHOULD include a URL that can be used to monitor the status of the request. lots of other existing implementations send one, including this plugin, p3k, webmention.io, and i think idno, so this probably isn't specific to bridgy. |
Either way, the filter addition allows for that and is probably better than
|
👍 filter (or hook) makes sense to me too. |
Thanks for pointing me to the response body part. I will also change the response of the plugin to return a more useful information, like the url of the comment. |
Use case for Bridgy....
"A POST to http://brid.gy/publish/webmention will return 200 OK on success with a JSON response containing at least a url field that points to the silo object that it operated on. For Twitter favorites and Facebook likes and event RSVPs, this is the tweet, post, or event. If a new object was created, e.g. a Facebook post or comment or Twitter tweet, @-reply, or retweet, there will also be an id field with the silo id of that object."
The json response can be extracted and the entire array attached to the post using the add_post_meta function in Wordpress.
This data can then be used by the theme or another plugin to add rel-syndication links.
The text was updated successfully, but these errors were encountered: