Page MenuHomePhabricator

GuidedTour external link buttons don't work
Closed, ResolvedPublicBUG REPORT

Description

Steps to replicate the issue (include links if applicable):

What happens?:

Nothing

What should have happened instead?:

The buttons should point somewhere.

Other information (browser name/version, screenshots, etc.):
These are defined by https://en.wikipedia.org/wiki/MediaWiki:Guidedtour-tour-twa1.js as action: 'externalLink',. The issue appears to be in GuidedTour itself not the local gadget and is a regression from https://gerrit.wikimedia.org/r/c/mediawiki/extensions/GuidedTour/+/982911 / T182050 by @Jdlrobson

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Thanks for the ping. I'll look into it.

Change #1028872 had a related patch set uploaded (by Pppery; author: Pppery):

[mediawiki/extensions/GuidedTour@master] Unbreak link buttons

https://gerrit.wikimedia.org/r/1028872

Change #1028872 merged by jenkins-bot:

[mediawiki/extensions/GuidedTour@master] Unbreak link buttons

https://gerrit.wikimedia.org/r/1028872

Pppery removed a project: Patch-For-Review.

Change #1030983 had a related patch set uploaded (by Jdlrobson; author: Pppery):

[mediawiki/extensions/GuidedTour@wmf/1.43.0-wmf.4] Unbreak link buttons

https://gerrit.wikimedia.org/r/1030983

Change #1030983 merged by jenkins-bot:

[mediawiki/extensions/GuidedTour@wmf/1.43.0-wmf.4] Unbreak link buttons

https://gerrit.wikimedia.org/r/1030983

Mentioned in SAL (#wikimedia-operations) [2024-05-13T21:10:32Z] <ebernhardson@deploy1002> Started scap: Backport for [[gerrit:1030983|Unbreak link buttons (T364062)]]

Mentioned in SAL (#wikimedia-operations) [2024-05-13T21:12:58Z] <ebernhardson@deploy1002> jdlrobson and ebernhardson: Backport for [[gerrit:1030983|Unbreak link buttons (T364062)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2024-05-13T21:32:32Z] <ebernhardson@deploy1002> Finished scap: Backport for [[gerrit:1030983|Unbreak link buttons (T364062)]] (duration: 22m 00s)

This has now been backported to production. Sorry for any disruption! cc @Quiddity

@Mrb_Rafi FYI, see the latest update above ("This has now been backported to production), because I saw you mention this bug in the hackathon telegram/IRC. This 'backport' effort from Jon means the fix should be live immediately (instead of waiting for the weekly deployment train to reach the Wikipedias on Thursday). I hope that helps a bit, and sorry the bug disrupted part of your event.
[Edit: And thank you, Jon, for working on the backport and review!]