RelatedArticles broken with other functionality - referral to no non-existent mobile.drawers
Closed, ResolvedPublic
Actions

Description

Caught by browser tests:
https://integration.wikimedia.org/ci/job/selenium-RelatedArticles/BROWSER=chrome,MEDIAWIKI_ENVIRONMENT=beta-mobile,PLATFORM=Linux,label=contintLabsSlave%20&&%20UbuntuTrusty/284/testReport/https://saucelabs.com/beta/tests/81048d5fe33c48898078c49fedb74ea0/commands#67

The console complains with
Uncaught Error: Unknown dependency: mobile.drawers

Let's not let this get to production...

Details

	Subject	Repo	Branch	Lines +/-
	Retrieve Drawer from its new home	mediawiki/extensions/Popups	master	+2 -2

Customize query in gerrit

Event Timeline

Jdlrobson created this task.Jan 23 2017, 7:02 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJan 23 2017, 7:02 PM

Jdlrobson moved this task from Needs Analysis to To Do on the Reading-Web-Sprint-90-🍌 board.Jan 23 2017, 7:04 PM

Jdlrobson renamed this task from RelatedArticles broken and breaks other functionality - refers no non-existent mobile.drawers to RelatedArticles broken with other functionality - referral to no non-existent mobile.drawers.Jan 23 2017, 7:08 PM

Jdlrobson triaged this task as Unbreak Now! priority.

Jdlrobson added projects: Page-Previews, RelatedArticles.

Restricted Application added subscribers: Jay8g, TerraCodes. · View Herald TranscriptJan 23 2017, 7:08 PM

Looks like this is probably a problem with the Popups master branch :-s

Change 333707 had a related patch set uploaded (by Jdlrobson):
Retrieve Drawer from its new home

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

gerritbot added a project: Patch-For-Review.Jan 23 2017, 7:10 PM

Dropping priority as I think this is feature flagged and only enabled on beta cluster - meaning it harms us testing on the beta cluster but shouldn't be an issue on production.