IPA is not properly removed in link previews
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	SUM1
	Nov 24 2017, 9:42 PM

Description

IPA templates (without brackets) are not properly removed in link previews. They leave an extra space.

Example: page – link preview

Example 2: page – link preview

Brackets, on the other hand, are properly removed, and no space is left.

Please fix this by setting Hovercards to remove the space before an IPA template.

Related Objects

Mentioned In: T113094: [EPIC] The Page Summary API needs to provide useful content for the majority of articles
Mentioned Here: T113094: [EPIC] The Page Summary API needs to provide useful content for the majority of articles

Event Timeline

SUM1 created this task.Nov 24 2017, 9:42 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 24 2017, 9:42 PM

We're in the process of moving this code to a new summary service which does not have this problem in T113094. Thanks for reporting this! It helps to have examples where the current summaries are broken!

Jdlrobson mentioned this in T113094: [EPIC] The Page Summary API needs to provide useful content for the majority of articles.Nov 27 2017, 6:01 PM

Jdlrobson moved this task from Untriaged to Untag on the Web-Team-Backlog (Tracking) board.Feb 7 2018, 5:51 PM

We should verify this is fixed. @ABorbaWMF can you help?

Jdlrobson moved this task from Incoming to QA Tasks / Needs Reproduction on the Web-Team-Backlog board.Feb 22 2018, 10:20 PM

It seems we can't test this on the beta cluster, but let's check it after we deploy html previews everywhere

ovasileva moved this task from Backlog to For Review on the Page-Previews board.Feb 26 2018, 4:05 PM

This can be tested on production now as the summary endpoint is live everywhere. We don't need to turn on html previews to test this as the bug report is based on text previews (also in the case of whitespace text and html are identical).

@Jdlrobson - you're right, I wasn't thinking. looks fixed: