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

Problem with Yoast Slack sharing: List Co Authors Plus authors in the Slack social share #952

Closed
Hercilio1 opened this issue Jul 25, 2023 · 5 comments · Fixed by #953
Closed

Comments

@Hercilio1
Copy link
Contributor

Hercilio1 commented Jul 25, 2023

The Slack social share still gets the native author.

Yoast has a feature that improves social share through Slack. This feature prints the author and the "time to read" of a post. Currently, as I said, it is not getting the Co-Authors Plus authors, but yes the native one.

This feature comes activated by default. You can check it out by going to the general settings of Yoast and looking for the following Social Share resource:
image

After making sure it is activated if you share through Slack a post with a native author that differs from the selected author (selected by Co-Authors block), you will see the native author.

@Hercilio1
Copy link
Contributor Author

You could use the hook wpseo_enhanced_slack_data to override the authors meta 😃

@GaryJones
Copy link
Contributor

@Hercilio1 Would you like to provide a PR for this, please? It seems like you're already familiar with the problem and possible solution.

@Hercilio1
Copy link
Contributor Author

@GaryJones , yes, no problem :)

Hercilio1 added a commit to Hercilio1/Co-Authors-Plus-Fork that referenced this issue Jul 26, 2023
- Regarding the issue Automattic#952, compatibility of the Co-Authors Plus with Yoast Slack sharing
@GaryJones
Copy link
Contributor

Self-note: Enhanced Slack Sharing was added to Yoast SEO 15.2 in October 2020.

@GaryJones GaryJones added this to the 3.5.12 milestone Jul 26, 2023
@GaryJones GaryJones linked a pull request Jul 27, 2023 that will close this issue
@GaryJones
Copy link
Contributor

Added with #953.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants