Open benlk opened 1 year ago
get_oembed_response_data is what provides the author and author_url keys there; it provides a filter oembed_response_data that allows us to change the response.
The problem is that it doesn't look like oEmbed supports multiple authorship: https://github.com/iamcal/oembed/issues/686
In the case of multiple bylines, since there's no support for multiple distinct authors and respective URLs, it may be easiest to do the following:
author_name
: use the CAP's byline generation function, then strip tagsauthor_url
: link to either the embedded page itself, or no URL@rebeccahum any thoughts from the team on an answer here or potential direction on any respective changes that could be contributed?
Can you give more detail on what is trying to be achieved here with steps on reproduction?
Thanks!
The goal here is to have Slack and other oEmbed consumers show the correct author(s) in the byline. It doesn't appear that that is the case with the current version of CAP.
Cool, feel free to open a PR for our review 🙂
Until the oembed spec changes (thanks for opening that issue), then I think the best we can do is make sure the author that is shown is the first / one of the Coauthors.
Here's a test post by a WP_User:
And here's a test post by a Coauthor: