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.

Sign up for GitHub

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

Jump to bottom

Fix: Robots tags conflicting with Yoast SEO. #53

Merged
merged 2 commits into from
Mar 25, 2021
Merged

Fix: Robots tags conflicting with Yoast SEO. #53

merged 2 commits into from
Mar 25, 2021

Conversation

AbhijitNage123
Copy link
Contributor

@AbhijitNage123 AbhijitNage123 commented Mar 24, 2021 •
edited by Nikschavan
Loading

Description

Screenshots

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

How has this been tested?

  • Yoast SEO plugin is present on the site and the page is selected to be crawled by Google and the page is also unlisted from our plugin then only add noindex and not add both index and noindex

Checklist:

  • My code is tested
  • My code passes the PHPCS tests
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

Nikschavan
Nikschavan requested changes Mar 24, 2021
View reviewed changes
Copy link
Collaborator

@Nikschavan Nikschavan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few minor changes in wordings, Looks ok otherwise

class-unlist-posts.php Show resolved Hide resolved
readme.txt Outdated Show resolved Hide resolved
@patilvikasj patilvikasj merged commit 28f0f96 into master Mar 25, 2021
@patilvikasj patilvikasj deleted the robot-tag-conflicting-with-yoast branch March 25, 2021 04:29
@Nikschavan Nikschavan mentioned this pull request Feb 21, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nikschavan Nikschavan Nikschavan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AbhijitNage123 @Nikschavan @patilvikasj