-
Notifications
You must be signed in to change notification settings - Fork 877
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
Use the CPT/taxonomy slug instead of the rewrite slug in the URLs for settings sections #20864
Comments
Please inform the customer of conversation # 1130682 when this conversation has been closed. |
In #20341 we noticed the same issue occurs for the Tutor LMS CPT Step-by-step reproduction instructions
Expected results |
The URL loaded by Yoast SEO for the three post types Assignments, Lessons and Quizzes is the same: A similar issue happens with the Directorist plugin #20016 |
Please inform the customer of conversation # 1072770 when this conversation has been closed. |
Issue merged from #20016 Issue: Custom taxonomy tags added by Directorist plugin failed to open its respective SEO settings. Step-by-step reproduction instructions
https://example.com/wp-admin/admin.php?page=wpseo_page_settings#/taxonomy/at_biz_dir-tags Expected results Custom post type category slug: at_biz_dir-category Custom post type tag slug: at_biz_dir-tags |
Please inform the customer of conversation # 996719 when this conversation has been closed. |
Please inform the customer of conversation # 1072770 when this conversation has been closed. |
Conversation # 1130682: Issue occurs with Paid Memberships Pro |
Please inform the customer of conversation # 1134067 when this conversation has been closed. |
Please inform the customer of conversation # 1136002 when this conversation has been closed. |
What is the goal of this issue?
We currently use the rewrite slugs instead of the post type or taxonomy slugs to build the direct URLs to the relevant sections in the settings, e.g.
/wp-admin/admin.php?page=wpseo_page_settings#/post-type/rewriteslug
instead of/wp-admin/admin.php?page=wpseo_page_settings#/post-type/posttypeslug
.This can be a problem in some cases since the rewrite slug can change, so the URLs would change too; also, only the main slug is unique, so if a site admin registers a post type or a taxonomy that overrides the rewrite slug of a builtin type, the section for that content type is unreachable.
This is already experienced by some users for taxonomies: #20016
What needs to be done to achieve the goal?
We need to update how the URLs are created to use the main slug and not the rewrite slug.
Does the issue still need UX or research?
Not really, we don't expect side effects (though we need to make sure to check this).
Being totally on the backend side, there should not be any effect on the front-end/SEO side
If available: what are the tips for fixing the problem or possible solutions?
See above
Should documentation be added or updated for this change? and if so, where?
No.
The text was updated successfully, but these errors were encountered: