-
Notifications
You must be signed in to change notification settings - Fork 1
AIOSEO integration #644
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
AIOSEO integration #644
Conversation
Implements AIOSEO equivalents for Yoast SEO recommendations #1-#8: - Archive Author: Disable author archives for single-author sites - Archive Date: Disable date archives when not used in permalinks - Archive Format: Disable format archives for sites with few formatted posts - Media Pages: Redirect media/attachment pages to parent posts - Crawl Settings Emoji Scripts: Enable crawl cleanup optimization - Crawl Settings Feed Authors: Disable author RSS feeds for single-author sites - Crawl Settings Feed Global Comments: Disable global comment RSS feeds - Organization Logo: Set organization or person logo for schema markup All providers follow the same pattern as Yoast integration and include proper condition checking using AIOSEO's options API. The integration automatically detects if All in One SEO is active via the aioseo() function. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
Simplified the check for author archives being disabled to use empty() which properly catches false, 0, '0', and null values. This ensures the task completes correctly when the user toggles the setting. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
Changed from strict comparison to loose comparison (! $show_value) to properly detect when the setting is false. This handles any type variations in how AIOSEO returns the value from its options API. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
Applied the same loose comparison fix as author archives. Uses ! $show_value instead of strict comparison to properly detect when date archives are disabled in AIOSEO settings. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
AIOSEO doesn't have a specific setting for post format archives like Yoast does. Removed the Archive_Format provider and its registration since this recommendation is not applicable to AIOSEO. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
Corrected the option path to searchAppearance->postTypes->attachment->redirectAttachmentUrls and updated task to specifically recommend 'attachment' redirect option. Task now completes properly when user sets redirect to 'attachment'. Updated title to be more specific about recommending attachment redirect. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
The redirectAttachmentUrls setting is stored in aioseo_options_dynamic, not the regular options. Updated to use aioseo()->dynamicOptions instead of aioseo()->options to properly access the attachment redirect setting. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> Co-authored-by: Sculptor <sculptor@imbue.com>
|
Test on Playground |
|
Did a first pass this morning, and at first glance it looks OK! |
|
Yeah an interactive task might be better, could you whip something up for that? |
|
@jdevalk , I have done it for Author and Global comments RSS feed tasks. To do it for the other as well (as far I can see Organization logo is the only one which actually require user to visit their page)? |
|
Yeah i think that's better! We might have to add a bit of text about why we suggest these things in the popup. Let me give you those: For comment feeds (where we disable just the global one now but we should actually disable
For author feeds:
For media pages:
|
|
And I forgot two: Author archives:
Date archives:
|
|
All done! I have also update the ID (and everything relared) for the |

Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
To test, install AIOSEO alongside this plugin.