WordPress.org

Making WordPress.org

Opened 8 months ago

Closed 8 months ago

#5360 closed defect (fixed)

Prevent broken /support/ links

Reported by: jonoaldersonwp Owned by:
Milestone: Priority: low
Component: International Sites (Rosetta) Keywords: seo
Cc:

Description

Many Rosetta sites link to a local /support/ URL (typically in the footer, but also from arbitrary in-content locations), but this page frequently doesn't exist and returns a 404.

E.g., https://de-ch.wordpress.org/support/.

This is a pretty terrible user experience, and represents many thousands of broken links throughout the ecosystem.

I propose that the best way to fix this would be to:

Whilst it's not ideal to run this kind of logic at this point in the cascade, it means that we can don't need to manage adding/removing redirects if individual locales create dedicated support pages.

Change History (5)

#1 @dd32
8 months ago

Do you have any examples of where things link to non-existent /support/ pages? We output a lot of localised-links and I wanted to make sure the place you're seeing this link isn't generated automatically incorrectly.

Returning a redirect for 404'ing /support/ seems reasonable to me.

#2 follow-up: @jonoaldersonwp
8 months ago

Sure!

Looks like all footers on plugin and theme pages (and /plugins/ and /themes/).

#3 in reply to: ↑ 2 @dd32
8 months ago

Replying to jonoaldersonwp:

Sure!

Looks like all footers on plugin and theme pages (and /plugins/ and /themes/).

Awesome, yup, that's a bug.
The footer on those pages should be the same as the homepage links.

The problem here is related to the wacky things we do to make the Themes/Plugins/Gutenberg WordPress.org subsites appear as a site in the Rosetta network too, as a result, the function responsible for finding out if de-ch.wordpress.org/support/ exists incorrectly checks for wordpress.org/support existing.

I'll get that fixed up so the links are correct, and get a redirect added anyway for ease of use.

#4 @jonoaldersonwp
8 months ago

Nice one! :)

#5 @dd32
8 months ago

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

Redirects in place:

curl -Is https://de-ch.wordpress.org/support/ | grep ^location
location: https://wordpress.org/support/

Forums redirects have a double redirect, but it's a convenience-only redirect anyway (It was already partially in place)

$ curl -ILs https://de-ch.wordpress.org/forums/ | grep ^location
location: https://de-ch.wordpress.org/support/
location: https://wordpress.org/support/

Footer links corrected:

$ curl -s https://de-ch.wordpress.org/plugins/hello-dolly/ https://de.wordpress.org/plugins/hello-dolly/ | grep -E '<li>.+?/support/'
<li><a href="//wordpress.org/support/" title="Forums, documentation, and other resources">Support</a></li>
<li><a href="//de.wordpress.org/support/" title="Foren, Dokumentationen und andere Ressourcen">Support</a></li>

(de has forum, de-ch does not)

Note: See TracTickets for help on using tickets.