Automattic/jetpack

Sitemaps: Disable Image Sitemaps

Open

#7,248 opened on May 26, 2017

View on GitHubContributor guide
 (18 comments) (3 reactions) (0 assignees)PHP (877 forks)user submission
Customer Report [LEGACY]EnhancementGood For CommunityHacktoberfest[Feature] Sitemaps[Pri] Normal[Type] Good First Bug

Repository metrics

Stars
 (1,808 stars)
PR merge metrics
 (Avg merge 4d 5h) (645 merged PRs in 30d)

Description

In the sitemaps module there's currently no way to disable any specific instances of the sitemaps, specifically the image sitemap.

Checking the developer options I'm seeing filters for excluding specific images but not a way to disable the full image sitemap.

requested in 3216712-t

Is there a way to turn off the image sitemaps entirely? We don't use attachment pages in WP, and the sitemap is loaded with attachment pages, since we have thousands of unattached photos. Obviously we don't want that stuff indexed. I poked around the code, but couldn't figure out an easy way to do that.

Contributor guide

Research direction
Locate the sitemap generation code in Jetpack, specifically the image sitemap provider class, and add a filter hook that allows disabling it entirely. Check existing filters like 'jetpack sitemap image skip post' and consider a new boolean filter such as 'jetpack disable image sitemap'.
Tech stack
php
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PHPWordPress
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65