Don't emit data-parsoid attributes for template content
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	ssastry
	Sep 12 2014, 8:48 PM

Description

Either strip this as part of the cleanup pass or don't save it during the saveDataParsoid pass, or some such thing.

Version: unspecified
Severity: normal

Mentioned In: rGPAR1dfabb84e70a: Merge "T72786: Re-enable stripping data-parsoid for template content"
rGPAR320caa300d2e: T72786: Re-enable stripping data-parsoid for template content

• bzimport raised the priority of this task from to Medium.Nov 22 2014, 3:57 AM

• bzimport set Reference to bz70786.

Change 160698 had a related patch set uploaded by Marcoil:
Bug #70786: Don't emit data-parsoid attributes for template content

Change 160698 merged by jenkins-bot:
Bug #70786: Don't emit data-parsoid attributes for template content

This has been disabled in production, pending further discussion with Parsoid users (like Flow) which could depend on data-parsoid being present.

• marcoil set Security to None.

AFAIK, the only blocker right now is the use of data-parsoid StructuredDiscussions makes.

(And if I got Phab correctly, mentioning them here should summon them from the darkness ;)

flow reads the data-parsoid for one case, which is solved in this parsoid patch: https://gerrit.wikimedia.org/r/#/c/170455/

Whenever that is merged you can go ahead and strip the data-parsoid attributes.

Change 178574 had a related patch set uploaded (by Marcoil):
T72786: Re-enable stripping data-parsoid for template content

Change 178574 merged by jenkins-bot:
T72786: Re-enable stripping data-parsoid for template content

Arlolra closed this task as Resolved.Dec 12 2014, 9:15 PM

Arlolra subscribed.