-
-
Notifications
You must be signed in to change notification settings - Fork 18
Add 'syndication' as link type to receive webmention. #31
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
base: master
Are you sure you want to change the base?
Conversation
Sites like https://indieweb.xyz and https://news.indieweb.org use a webmention sent for a link with "u-syndication" indicated, see: https://indieweb.xyz/howto/en and https://news.indieweb.org/how-to-submit-a-post It's possible that this would be a good moment to reexamine whether there are other classes of link that merit inclusion here, but this change is limited to what I know would be useful for a user of the service (i.e., me).
Thanks for sending this through. We you able to test the code with this code or is it a change in theory? |
A change in theory alone
…On Sun, Jul 19, 2020, 3:46 AM Remy Sharp ***@***.***> wrote:
Thanks for sending this through. We you able to test the code with this
code or is it a change in theory?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#31 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEOINLFPOLWOLUS4TI5AQ6LR4LFG7ANCNFSM4PAQ2RXA>
.
|
With change:
without change:
|
Originally the body of my post linked to indieweb.xyz which caused me some confusion because this line selected the body link not the syndication link, so the body link did not have the webmention endpoint attached to it. Separate issue, though. |
Let me know if there's anything else you need here. |
I noticed this as well while trying to automate sending webmentions to brid.gy. As mentioned in #50, if the syndication links are inside of Expected working version <article class="h-entry">
<details hidden>
<div class="p-author h-card"><!-- ... --></div>
<a class="u-url" href="/notes/1667940289/"></a>
<a rel="syndication noreferrer" class="u-syndication" href="https://brid.gy/publish/mastodon"></a>
</details>
<div class="e-content">
<p><!-- ... --></p>
</div>
<footer><!-- ... --></footer>
</article> Current working version <article class="h-entry">
<details hidden>
<div class="p-author h-card"><!-- ... --></div>
<a class="u-url" href="/notes/1667939162/"></a>
</details>
<div class="e-content">
<p><!-- ... --></p>
<details hidden><a rel="syndication noreferrer" class="u-syndication" href="https://brid.gy/publish/mastodon"></a></details>
</div>
<footer><!-- ... --></footer>
</article> |
In a HWC, someone recommended sending mentions for all |
Sites like https://indieweb.xyz and
https://news.indieweb.org use a webmention sent
for a link with "u-syndication" indicated, see:
https://indieweb.xyz/howto/en
and
https://news.indieweb.org/how-to-submit-a-post
It's possible that this would be a good moment
to reexamine whether there are other classes of
link that merit inclusion here, but this change
is limited to what I know would be useful for a
user of the service (i.e., me).
I think this might be my first Github PR? Please let me know if there's anything I can do to make this an acceptable change.