Make WordPress Core

Opened 8 weeks ago

Closed 8 weeks ago

Last modified 5 weeks ago

#63069 closed task (blessed) (fixed)

No index readme.html

Reported by: peterwilsoncc's profile peterwilsoncc Owned by: audrasjb's profile audrasjb
Milestone: 6.8 Priority: normal
Severity: normal Version:
Component: General Keywords: good-first-bug has-patch commit has-dev-note
Focuses: Cc:

Description

A search for text in the WordPress readme.html file shows a lot of sites with the page showing up in their search results (you will need to click "show duplicate results".

In all likelihood this is unintended by the site owner as the content is unrelated to their site. It also affects the SEO results of the install guide on WordPress.org.

To prevent this, it would be beneficial to add a noindex,nofollow meta tag to the header of this page.

As this is a small change, I've put it on the 6.8 milestone as a task but I'm happy if the release squad would prefer to wait until the next release.

Follow up to #63000.

Change History (9)

#1 @sabernhardt
8 weeks ago

  • Keywords needs-patch good-first-bug added

@dd32 also mentioned this in a Slack discussion.

This ticket was mentioned in PR #8471 on WordPress/wordpress-develop by @abcd95.


8 weeks ago
#2

  • Keywords has-patch added; needs-patch removed

#3 @audrasjb
8 weeks ago

The above PR correctly adds a meta tag in the readme.html file, but what about adding a directive in the virtual robots.txt file?

#4 @audrasjb
8 weeks ago

  • Keywords commit added
  • Owner set to audrasjb
  • Status changed from new to accepted

Hmm. Second thought: Given we're in the beta cycle and as this is in fact an enhancement, let's keep it in a limited scope and a self-contained, low-risk change :)

A follow-up ticket can always be created later to update the virtual robots.txt file, if needed.

#5 @audrasjb
8 weeks ago

  • Keywords needs-dev-note added

Note: This will need a Make/Polyglots post to let Locales know that the readme file was modified. Indeed, some historic locales (such as fr_FR) are managing their own version of the readme.html file.
Example: https://i18n.svn.wordpress.org/fr_FR/trunk/dist/

I'm adding the needs-dev-note workflow keyword as a reminder that we'll need a note on Make/Polyglots P2.

#6 @audrasjb
8 weeks ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 59949:

General: Add noindex,nofollow meta tag to the readme.html file.

Site owners likely don't intend for the content of the readme.html file to be indexed, as it's unrelated to the site content.

Follow-up to [59944].

Props peterwilsoncc, sabernhardt, dd32, audrasjb, abcd95.
Fixes #63069.
See #63000.

@abcd95 commented on PR #8471:


7 weeks ago
#7

tickets with keyword good-first-bug should be kept for new contributors so they can learn to contribute to WordPress Core. It doesn't look like you are a new contributor :)

Understood @audrasjb 😃

#9 @JeffPaul
5 weeks ago

  • Keywords has-dev-note added; needs-dev-note removed

This has now been published as a section in the Miscellaneous developer changes in WordPress 6.8 dev note post.

Note: See TracTickets for help on using tickets.