Page MenuHomePhabricator

Parsoid REST Routes Extension
Closed, ResolvedPublic


Based on our discussion in T224471, we need an extension in the Parsoid repo to implement the routes. Per @Anomie in T224471#5218407:

  • "Develop the code in some other repo, and copy to MediaWiki core later.
    • Could be arbitrarily in a subdirectory of Parsoid-PHP (as suggested in T224471#5216381), or could be in a temporary extension (suggested in the Parsoid meeting today).
      • Let's put "Temporary" in the extension name, so as to not let anyone start to think it's not temporary.
  • Development would involve pulling in the code from the other repo in some manner. As an extension that would probably be extremely straightforward (we need to write that code anyway), as an arbitrary subdirectory it might be less so. OTOH, the arbitrary subdirectory could be exactly what would be in an extension anyway, just not as a separate repo, which would have the advantage of fewer cross-repo changes.
  • History might be lost (or only included by reference in the commit summary), or might be like #2, depending on how the eventual merge-into-core happens.

After writing all that, I'm liking #5 so far, specifically the version of structuring a subdirectory in the Parsoid-PHP repo as a MW extension, followed by #2. #4 seems like too much work that will be thrown away, and #3 seems like rebase hell."

(Numbers refer to other options in that comment. We decided on this option.)

Event Timeline

Change 514356 had a related patch set uploaded (by Gergő Tisza; owner: Gergő Tisza):
[mediawiki/services/parsoid@master] Turn Parsoid into an extension temporarily

Change 514356 merged by jenkins-bot:
[mediawiki/services/parsoid@master] Turn Parsoid into an extension temporarily