Page MenuHomePhabricator

Update Flow for Parsoid changes re data-mw
Open, Needs TriagePublic

Description

See blocking tasks.

Event Timeline

ssastry raised the priority of this task from to Needs Triage.
ssastry updated the task description. (Show Details)
ssastry subscribed.
Restricted Application added subscribers: StudiesWorld, Aklapper. · View Herald Transcript

I was under the impression that Flow stored HTML, and needs to convert to wikitext for editing. Is that still the case?

The stashing functionality in RB is useful to preserve wikitext when temporarily switching to HTML editing. Does this apply here?

The stashing functionality in RB is useful to preserve wikitext when temporarily switching to HTML editing. Does this apply here?

I was under the impression that stashing was the exact feature lacking in RB for Flow to adopt it.

I was under the impression that Flow stored HTML, and needs to convert to wikitext for editing. Is that still the case?

Yes (although the HTML can also be edited with VE).

Okay, so it sounds like the issue is really getting data-parsoid and data-mw from the wt2html end point, and storing it along the HTML in Flow storage.

I see two main options:

1) Provide pagebundle end points for wt2html and html2wt.

These are JSON blobs containing data-parsoid, html & later data-mw, and could be stored in Flow storage.

Difficulty on the RESTBase side: easy.
Difficulty on the Flow side: easy-moderate; does not solve change propagation.

2) Move Flow storage to RESTBase, as discussed in T94574.

While the storage itself is fairly straightforward, we would not be able to use much of the Parsoid logic due to its assumption that content is available on-wiki. The change propagation issues mentioned on the task are a separate issue altogether.

Difficulty on RESTBase side: Moderate-complex, especially change propagation.
Difficulty on Flow side: moderate?

Mattflaschen-WMF renamed this task from Migrate Flow to talk with RESTBase instead of Parsoid to Update Flow for Parsoid changes re data-mw.EditedFeb 4 2016, 6:26 PM
Mattflaschen-WMF edited projects, added Parsoid; removed RESTBase.
Mattflaschen-WMF set Security to None.

After meeting, we decided to still talk to Parsoid: https://etherpad.wikimedia.org/p/Flow-Restbase-2016-02-04

JJMC89 raised the priority of this task from High to Needs Triage.May 18 2018, 5:46 PM
SBisson claimed this task.
SBisson subscribed.

No activity for 2.5 years. I assume this is not a problem anymore.

No, this will need to be done when (not if) data-mw split happens, unless we explicitly resolve we will maintain a separate flag / endpoint for Flow to preserve the old format.

@ssastry My bad. Do you have an idea when that will be needed?

@ssastry My bad. Do you have an idea when that will be needed?

Not yet .. we've delayed doing this till it gets close to when we absolutely have to do this. We'll provide sufficient advance notice before we actually do this.

@ssastry Thanks. I'll put it in "future" until you tell us it is needed.