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

Non Indexed Tabs should not be in the SiteMap #3896

Closed
ajwaka opened this issue Jul 6, 2020 · 2 comments · Fixed by #3897
Closed

Non Indexed Tabs should not be in the SiteMap #3896

ajwaka opened this issue Jul 6, 2020 · 2 comments · Fixed by #3897

Comments

@ajwaka
Copy link
Contributor

ajwaka commented Jul 6, 2020

Ideally - a sitemap tells a crawler what pages to scan and index. Therefore, any tab that is not to be indexed should NOT be included in the sitemap output.

The issue is then compounded if the Priority is kept at 0.5. Because the sitemap states to crawl this page and and it means something, by being set as to not index - it's also telling the search engine to ignore it. Google is negatively affecting rankings due to this issue

@ajwaka
Copy link
Contributor Author

ajwaka commented Jul 8, 2020

The submitted fix was using the tab's "Indexed" property, which is actually referring to a ContentItem. Not the proper TabSettings value for "AllowIndex". I have a new submission to properly fix the issue.

@ajwaka
Copy link
Contributor Author

ajwaka commented Jul 8, 2020

Submitted actual fix - sorry for the confusion. We're working on a fix for all of our older sites when the mis-assumption of tab.Indexed was built in.

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 a pull request may close this issue.

1 participant