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

Remove faulty integration with Autoshare for Twitter #221

Open
wants to merge 1 commit into
base: trunk
Choose a base branch
from

Conversation

elvismdev
Copy link

This hook in the Syndication Links plugin creates a conflict with the ongoing Autoshare for X + Syndication Links integration currently in progress at 10up/autoshare-for-twitter#331.

Incomplete Tweet URLs, such as https://twitter.com/i/web/status/, are being generated and added because the Syndication Links plugin hooks into autoshare_for_twitter_after_status_update to handle the same functionality being developed in the Autoshare for Twitter plugin PR.

Additionally, this implementation contains a bug in the variable names – it attempts to use the undefined variable $tweet_status, leaving $data unused.

Finally, by using add_post_syndication_link( $post->ID, $url ), it does not account for saving syndication link metadata in custom post types. This contrasts with using \Syn_Meta::add_syndication_link( $type, $post_id, $uri );, which allows specifying the post type object where the metadata should be saved through the $type parameter.

@dshanske
Copy link
Owner

dshanske commented Jan 6, 2025

This is great... I will dig in in further detail and review.

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.

2 participants