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

Mf2 support #17

Merged
merged 55 commits into from
Feb 19, 2014
Merged

Mf2 support #17

merged 55 commits into from
Feb 19, 2014

Conversation

mapkyca
Copy link

@mapkyca mapkyca commented Feb 19, 2014

Endpoint now tries to parse the endpoint for MF2 before falling back to the current method. Also changed the duplicate comment detection.

This lets the webmention more accurately render as a comment.

Also saved the profile icon url against the comment, so themes can use this (or ideally the plugin hooks in and provides this where present, when I work out how)

pfefferle and others added 30 commits July 31, 2013 08:44
the plugin should only implement the spec stuff
finger->mention
finger->mention
works for me with 3.8, modulo bugs reported, which should have no bearing on wordpress version at least up to 3.8.
and grunt task to convert readme.txt to readme.md
thanks to @snarfed for the ideas
* added post-id to `webmention_links` filter
* improved `publish_post_hook` function
@pfefferle
Copy link
Owner

Oh man! You have done a lot of work, thanks a lot for that!

...I hope you were aware of the fact, that we only moved the mf2 stuff to another plugin because it is not really part of the webmention spec? https://github.com/pfefferle/wordpress-semantic-linkbacks

pfefferle added a commit that referenced this pull request Feb 19, 2014
@pfefferle pfefferle merged commit f158040 into pfefferle:mf2-support Feb 19, 2014
@mapkyca
Copy link
Author

mapkyca commented Feb 19, 2014

Happy to help!

I was vaguely aware that it wasn't part of the spec, although it seems that most implementations use it, so it made sense to try.

Btw, just figured out how to take advantage of the icon url we've saved, so I've pushed that too..

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.

4 participants