Page MenuHomePhabricator

Add parameter that allows language button to be seen across all wikis
Closed, ResolvedPublic2 Estimated Story Points

Description

Background

We would like to flag potential issues on the pilot wikis for the desktop improvements project with the new language button. One way to do this would be to add a parameter that shows the language button across all wikis

Acceptance criteria

  • explore adding a parameter that shows the language button across all wikis regardless of whether the A/B test is enabled

Notes

  • There's already a check for a query parameter in the Vector\FeatureManagement\Requirements\LanguageInHeaderTreatmentRequirement::isMet() method but it's dependent on a check for whether the A/B test is enabled. The former needs to be independent of the latter.

QA

  1. Go to https://en.wikipedia.beta.wmflabs.org/wiki/Dog?languageinheader=0
  2. Verify that the language list shows up in the sidebar
  1. Go to https://en.wikipedia.beta.wmflabs.org/wiki/Dog?languageinheader=1
  2. Verify that the language list shows up in the content header (next to "Dog")

QA Results - Beta

ACStatusDetails
1T282543#7099728
2T282543#7099728

QA Results - Prod

Event Timeline

I believe that this is already done. Add languageinheader=1 to the query string.

I believe that this is already done. Add languageinheader=1 to the query string.

Is there any way we can get it to work if the feature's not available on a wiki yet?

I think the main use case here is getting the button to show on https://fr.wikipedia.org/wiki/Paris for QA prior to deployment

Change 690620 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@master] Allow `languageinheader` query param to fully control treatment of languages

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

nray removed nray as the assignee of this task.May 13 2021, 6:03 PM
nray added a subscriber: nray.

Change 690620 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Allow `languageinheader` query param to fully control treatment of languages

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

Change 692075 had a related patch set uploaded (by Nray; author: Nray):

[mediawiki/skins/Vector@wmf/1.37.0-wmf.5] Allow `languageinheader` query param to fully control treatment of languages

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

Change 692075 merged by jenkins-bot:

[mediawiki/skins/Vector@wmf/1.37.0-wmf.5] Allow `languageinheader` query param to fully control treatment of languages

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

Mentioned in SAL (#wikimedia-operations) [2021-05-17T18:27:35Z] <urbanecm@deploy1002> Synchronized php-1.37.0-wmf.5/skins/Vector/includes/FeatureManagement/Requirements/LanguageInHeaderTreatmentRequirement.php: e180b99: Allow languageinheader query param to fully control treatment of languages (T282543) (duration: 00m 58s)

Can confirm parameter is working on pilot wikis, @Edtadros - not sure if any additional testing is needed here. A couple of more notes:

frwiki

Screen Shot 2021-05-19 at 4.51.20 PM.png (984×2 px, 653 KB)

fawiki

  • Issues with coordinate templates
  • Other indicators (lock, featured) seem okay

Screen Shot 2021-05-19 at 4.58.23 PM.png (1×2 px, 1 MB)

ptwiki

  • Looks okay with all indicators

Screen Shot 2021-05-19 at 5.02.25 PM.png (1×2 px, 1 MB)

euwiki

  • Looks okay with all indicators

Screen Shot 2021-05-19 at 5.05.59 PM.png (1×2 px, 1 MB)

frwiktionary

Screen Shot 2021-05-19 at 5.08.02 PM.png (1×2 px, 632 KB)

hewiki

  • Seems okay with indicators & coordinates
  • Can't find an example that has both

Screen Shot 2021-05-19 at 5.10.37 PM.png (1×2 px, 774 KB)

Screen Shot 2021-05-19 at 5.09.26 PM.png (1×2 px, 1006 KB)

bnwiki

  • Issues with coordinate templates & indicators

Screen Shot 2021-05-19 at 5.13.23 PM.png (1×2 px, 1 MB)

dewikivoyage

  • Looking good!

Screen Shot 2021-05-19 at 5.18.06 PM.png (1×2 px, 1 MB)

kowiki:

  • Issues with coordinates
  • Indicators are ok

Screen Shot 2021-05-19 at 5.19.59 PM.png (1×2 px, 1 MB)

srwiki:

  • Issues with coordinates
  • Indicators ok

Screen Shot 2021-05-19 at 5.21.57 PM.png (1×2 px, 1 MB)

trwiki:

  • Issues with coordinates and indicators
  • Language button note appearing at all

Screen Shot 2021-05-19 at 5.22.48 PM.png (1×2 px, 1 MB)

vecwiki:

  • Looks okay

Screen Shot 2021-05-19 at 5.24.42 PM.png (1×2 px, 1 MB)

@ovasileva you're very thorough! I will test the same wikis with the URL parameters. Beta works.

Status: ✅ PASS
Environment: beta
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

✅ AC1: Go to https://en.wikipedia.beta.wmflabs.org/wiki/Dog?languageinheader=0
Verify that the language list shows up in the sidebar

Screen Shot 2021-05-19 at 5.50.13 PM.png (1×1 px, 596 KB)

✅ AC2: Go to https://en.wikipedia.beta.wmflabs.org/wiki/Dog?languageinheader=1
Verify that the language list shows up in the content header (next to "Dog")

Screen Shot 2021-05-19 at 5.50.40 PM.png (1×1 px, 428 KB)

Edtadros added a subscriber: Edtadros.

Status: ✅ PASS
Environment: various
OS: macOS Big Sur
Browser: Chrome
Device: MBP
Emulated Device: NA

Test Artifact(s):

QA Steps

✅ AC1: Go to a page in each wiki below using the following URL parameter: ?languageinheader=0
Verify that the language list shows up in the sidebar

frwikifawikiptwikieuwikifrwiktionaryhewikibnwikidewikivoyagekowikisrwikitrwikivecwiki
Screen Shot 2021-05-19 at 5.58.03 PM.png (1×1 px, 911 KB)
Screen Shot 2021-05-19 at 6.01.17 PM.png (1×1 px, 636 KB)
Screen Shot 2021-05-19 at 6.02.17 PM.png (1×1 px, 658 KB)
Screen Shot 2021-05-19 at 6.03.25 PM.png (1×1 px, 713 KB)
Screen Shot 2021-05-19 at 6.04.50 PM.png (1×1 px, 507 KB)
Screen Shot 2021-05-19 at 6.22.16 PM.png (1×1 px, 713 KB)
Screen Shot 2021-05-19 at 6.09.45 PM.png (1×1 px, 648 KB)
Screen Shot 2021-05-19 at 6.11.55 PM.png (1×1 px, 850 KB)
Screen Shot 2021-05-19 at 6.13.37 PM.png (1×1 px, 660 KB)
Screen Shot 2021-05-19 at 6.14.55 PM.png (1×1 px, 531 KB)
Screen Shot 2021-05-19 at 6.16.48 PM.png (1×1 px, 470 KB)
Screen Shot 2021-05-19 at 6.19.40 PM.png (1×1 px, 714 KB)

✅ AC2: Go to a page in each wiki below using the following URL parameter: ?languageinheader=1
Verify that the language list shows up in the content header (next to "Dog")

frwikifawikiptwikieuwikifrwiktionaryhewikibnwikidewikivoyagekowikisrwikitrwikivecwiki
Screen Shot 2021-05-19 at 5.58.18 PM.png (1×1 px, 884 KB)
Screen Shot 2021-05-19 at 6.01.28 PM.png (1×1 px, 617 KB)
Screen Shot 2021-05-19 at 6.02.26 PM.png (1×1 px, 794 KB)
Screen Shot 2021-05-19 at 6.03.46 PM.png (1×1 px, 689 KB)
Screen Shot 2021-05-19 at 6.05.11 PM.png (1×1 px, 497 KB)
Screen Shot 2021-05-19 at 6.22.31 PM.png (1×1 px, 704 KB)
Screen Shot 2021-05-19 at 6.10.01 PM.png (1×1 px, 628 KB)
Screen Shot 2021-05-19 at 6.12.11 PM.png (1×1 px, 827 KB)
Screen Shot 2021-05-19 at 6.13.53 PM.png (1×1 px, 645 KB)
Screen Shot 2021-05-19 at 6.15.07 PM.png (1×1 px, 516 KB)
Screen Shot 2021-05-19 at 6.17.03 PM.png (1×1 px, 451 KB)
Screen Shot 2021-05-19 at 6.19.49 PM.png (1×1 px, 714 KB)

@ovasileva your testing was more than enough. I thought I saw some wikis where the new language button wasn't showing up so I decided to test it, but after I finished this I realized it was because it was old vector. Oops.