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

noindex/nofollow for no SEO on post type only works together #1957

Closed
michaeltorbert opened this issue Oct 8, 2018 · 1 comment
Closed
Assignees
Labels
Milestone

Comments

@michaeltorbert
Copy link
Contributor

michaeltorbert commented Oct 8, 2018

For #1797 and #1943 we made it so noindex/nofollow works when SEO for that post type is disabled, however this only works if you're using both noindex and nofollow. If you're trying to use just one it doesn't work.

To make the logic easier, it may be best to pull this out into its own function(s). Eventually it should be in its own file, rather than junking up that big class file.

@EkoJR
Copy link
Contributor

EkoJR commented Oct 9, 2018

It seems the git merge didn't go as smooth as would of hoped; looks like it duplicated code instead of line changes.

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

No branches or pull requests

2 participants