You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It'd be nice, I guess, if the plugin took a guess and assumed this being in reply to another comment, and use the URL fragment to correctly nest the mention?
In IndieBlocks, I've attempted to do just that.
In Webmention, I noticed this bit of code that seems to store the fragment but not use it?
Should I be using ?replytocom=<ID> instead? (I think we've discussed this before but I couldn't seem to find it back immediately.) Because that's not a URL that can readily be found. The "comment u-url" is the one with the fragment part, after all. (But I'm okay with adding the replytocom query argument as well, to improve compatibility.)
The text was updated successfully, but these errors were encountered:
We discussed that for ActivityPub (which should not work because fragments are ignored by the server, so content negotiation wouldn't work), but I like the idea for Webmentions. Will work on an implementation!
I just published this reply to https://nicksimson.com/notes/941927cc48/#comment-782 (note the URL fragment), and noticed it didn't quite get "nested" correctly, at least not automatically.
It'd be nice, I guess, if the plugin took a guess and assumed this being in reply to another comment, and use the URL fragment to correctly nest the mention?
In IndieBlocks, I've attempted to do just that.
In Webmention, I noticed this bit of code that seems to store the fragment but not use it?
https://github.com/pfefferle/wordpress-webmention/blob/main/includes/class-receiver.php#L281-L298
Should I be using
?replytocom=<ID>
instead? (I think we've discussed this before but I couldn't seem to find it back immediately.) Because that's not a URL that can readily be found. The "commentu-url
" is the one with the fragment part, after all. (But I'm okay with adding thereplytocom
query argument as well, to improve compatibility.)The text was updated successfully, but these errors were encountered: