Maniphest T362876

Mobile-html doesn't render the expected leading paragraph
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	Jgiannelos
	Thu, Apr 18, 12:43 PM

Tags

Referenced Files

	F47333154: image.png
	Thu, Apr 18, 1:00 PM

Subscribers

Description

Detected from CI failure:
https://gerrit.wikimedia.org/r/c/mediawiki/services/mobileapps/+/1021442

Details

	Subject	Repo	Branch	Lines +/-
	Fix broken lead section paragraph	mediawiki/services/mobileapps	master	+64 -6

Customize query in gerrit

Event Timeline

Jgiannelos created this task.Thu, Apr 18, 12:43 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptThu, Apr 18, 12:43 PM

Jgiannelos added projects: Page Content Service, Content-Transform-Team-WIP.Thu, Apr 18, 12:44 PM

From a quick look this is the <p> element that its chosen as a lead paragraph instead of the actual first paragraph:

From:
https://en.wikipedia.org/api/rest_v1/page/html/Dog/844680047

//*[@id="mwBg"]

Change #1021504 had a related patch set uploaded (by Jgiannelos; author: Jgiannelos):

[mediawiki/services/mobileapps@master] Fix broken lead section paragraph

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

gerritbot added a project: Patch-For-Review.Thu, Apr 18, 3:24 PM

Jgiannelos added a project: Essential-Work.Thu, Apr 18, 3:33 PM

Jgiannelos moved this task from Backlog to Code Review on the Content-Transform-Team-WIP board.

Change #1021504 merged by jenkins-bot:

[mediawiki/services/mobileapps@master] Fix broken lead section paragraph

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

Jgiannelos moved this task from Code Review to To Deploy on the Content-Transform-Team-WIP board.Tue, Apr 23, 12:26 PM

Jgiannelos moved this task from To Deploy to To Verify on the Content-Transform-Team-WIP board.Tue, Apr 23, 1:36 PM

Jgiannelos closed this task as Resolved.Tue, Apr 23, 3:27 PM

Jgiannelos claimed this task.

Maintenance_bot removed a project: Patch-For-Review.Fri, Apr 26, 5:52 PM