| Jdlrobson | |
| Mar 12 2024, 4:43 PM |
| F42655670: screenshot.mov.gif | |
| Mar 15 2024, 10:31 PM |
Steps to replicate the issue (include links if applicable):
What happens?:
Appears in dark mode
What should have happened instead?:
Appears in light mode with an notification
Software version (skip for WMF-hosted wikis like Wikipedia):
Other information (browser name/version, screenshots, etc.):
| AC | Status | Details |
|---|---|---|
| 1 | ✅ | T359958#9635161 |
| 2 | ✅ | T359958#9635194 |
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | ovasileva | T341631 [EPIC] Q1 Main hypothesis: Typographical and palette customizations | |||
| Resolved | ovasileva | T341632 [EPIC] Palette customizations and dark mode | |||
| Resolved | ovasileva | T354835 [epic] Release night mode as a beta feature on desktop and mobile | |||
| Resolved | ovasileva | T359152 Deploy initial version of night mode to pilot wikis on the mobile website for testing | |||
| Resolved | ovasileva | T359183 Exclude non-functional pages from night mode | |||
| Resolved | BUG REPORT | ovasileva | T359958 Exclude list is not working for special page names |
Change 1010257 had a related patch set uploaded (by Mabualruz; author: Mabualruz):
[mediawiki/skins/MinervaNeue@master] Fix Issue with localisation of special page titles in exclusion logic
Change 1010697 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/skins/MinervaNeue@master] Fix Issue with localization of special page titles in exclusion logic
Change 1010257 had a related patch set uploaded (by Jdlrobson; author: Mabualruz):
[mediawiki/skins/MinervaNeue@master] Fix Issue with localization of special page titles in exclusion logic
Change 1010257 abandoned by Mabualruz:
[mediawiki/skins/MinervaNeue@master] Fix Issue with localization of special page titles in exclusion logic
Reason:
solution in 1010697
Change 1010697 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@master] Fix Issue with localization of special page titles in exclusion logic
Change 1010585 had a related patch set uploaded (by Jdlrobson; author: Jdlrobson):
[mediawiki/skins/MinervaNeue@wmf/1.42.0-wmf.22] Fix Issue with localization of special page titles in exclusion logic
Change 1010585 merged by jenkins-bot:
[mediawiki/skins/MinervaNeue@wmf/1.42.0-wmf.22] Fix Issue with localization of special page titles in exclusion logic
Mentioned in SAL (#wikimedia-operations) [2024-03-13T20:22:28Z] <jdrewniak@deploy2002> Started scap: Backport for [[gerrit:1010585|Fix Issue with localization of special page titles in exclusion logic (T359958)]]
Mentioned in SAL (#wikimedia-operations) [2024-03-13T20:24:50Z] <jdrewniak@deploy2002> jdlrobson and jdrewniak: Backport for [[gerrit:1010585|Fix Issue with localization of special page titles in exclusion logic (T359958)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)
Mentioned in SAL (#wikimedia-operations) [2024-03-13T20:39:17Z] <jdrewniak@deploy2002> Finished scap: Backport for [[gerrit:1010585|Fix Issue with localization of special page titles in exclusion logic (T359958)]] (duration: 16m 49s)
Status: ❌ FAIL
Environment: frwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NA
Test Artifact(s):
✅ AC1: Appears in light mode
❌ AC2: with an notification
There is no notification
@Edtadros the notifications cannot be tested on right now. We should repeat this test as part of the deployment Monday.
The screenshot also seems incorrect?
Test Result - Prod
Status: ❌ FAIL
Environment: frwiki
OS: macOS Sonoma
Browser: Chrome
Device: MBA
Emulated Device:NATest Artifact(s):
QA Steps
✅ AC1: Appears in light mode
❌ AC2: with an notification
There is no notification
The notification translates to:
Unfortunately, grouping results by page is not available on mobile. If you would like to use the page grouping feature, please visit the desktop site.
https://fr.m.wikipedia.org/w/index.php?title=Sp%C3%A9cial:Modifications_r%C3%A9centes&days=30&enhanced=1&hideWikibase=1&hidebots=1&hidecategorization=1&limit=500&safemode=1&urlversion=2 even without the night mode parameter added the notification persists
I do not think it relates to the ticket. @Jdlrobson please advise