Way for VE to run on Flow without changing wgVisualEditorNamespaces
Closed, ResolvedPublic0 Estimated Story Points
Actions

Description

Changing the VisualEditorNamespaces config (wgVisualEditorNamespaces) is currently required to use VisualEditor with Flow. However, this has other undesired side effects, such as marking VE available in ve.init.mw.ViewPageTarget.init.js.

Also, if Flow were not enabled on certain pages in that namespace, normal VE would then be enabled there, which is undesirable.

There should be a way for us to use the API (or certain safe parts of it, such as 'parsefragment') in certain namespaces without the other side effects.

Details

Project	Branch	Lines +/-	Subject
mediawiki/extensions/VisualEditor	master	+11 -1	Allow parsefragment even if VE is not enabled for that title
mediawiki/extensions/VisualEditor	wmf/1.25wmf24	+11 -1	Allow parsefragment even if VE is not enabled for that title
operations/mediawiki-config	master	+3 -1	Add NS_TALK to VE for Flow, only on MW.org, test, and test2.

Customize query in gerrit

Revisions and Commits

rEVED extension-VisualEditor-MediaWiki
	rEVEDe194a6fd36f5 Allow parsefragment even if VE is not enabled for that title
	rEVEDc6ea5d44a7a7 Allow parsefragment even if VE is not enabled for that title
	rEVEDbe9847c24f23 Allow parsefragment even if VE is not enabled for that title

Related Objects

Mentioned In: rOMWCd5bc66d4920e: Revert "Enable VisualEditor for Flow posts on Beta Labs"

Event Timeline

• Mattflaschen-WMF created this task.Mar 27 2015, 11:03 PM

• Mattflaschen-WMF raised the priority of this task from to Needs Triage.

• Mattflaschen-WMF updated the task description. (Show Details)

• Mattflaschen-WMF added projects: StructuredDiscussions, VisualEditor.

• Mattflaschen-WMF added a subscriber: • Mattflaschen-WMF.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 27 2015, 11:03 PM

Change 201388 had a related patch set uploaded (by Mattflaschen):
Add NS_TALK to VE for Flow, only on MW.org, test, and test2.

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

Restricted Application added a project: Collaboration-Team-Triage. · View Herald TranscriptApr 1 2015, 11:55 PM

gerritbot added a project: Patch-For-Review.Apr 1 2015, 11:55 PM

Does anyone know why https://git.wikimedia.org/blob/mediawiki%2Fextensions%2FVisualEditor.git/792e1444de3d2ba2b2902c450252a230924902e8/ApiVisualEditor.php#L269 is there?

It doesn't seem necessary, at least for 'parsefragment'.

• Mattflaschen-WMF updated the task description. (Show Details)Apr 2 2015, 12:10 AM

• Mattflaschen-WMF set Security to None.

Change 201388 abandoned by Mattflaschen:
Add NS_TALK to VE for Flow, only on MW.org, test, and test2.

Reason:
This was only meant to be temporary, but okay.

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

• Mattflaschen-WMF claimed this task.Apr 2 2015, 12:22 AM

Change 201400 had a related patch set uploaded (by Mattflaschen):
Allow parsefragment even if VE is not enabled for that title

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

Change 201488 had a related patch set uploaded (by Jforrester):
Allow parsefragment even if VE is not enabled for that title

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

Change 201400 merged by jenkins-bot:
Allow parsefragment even if VE is not enabled for that title

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

• Mattflaschen-WMF closed this task as Resolved by committing rEVEDbe9847c24f23: Allow parsefragment even if VE is not enabled for that title.Apr 2 2015, 4:58 PM

• Mattflaschen-WMF added a commit: rEVEDbe9847c24f23: Allow parsefragment even if VE is not enabled for that title.

Change 201488 merged by jenkins-bot:
Allow parsefragment even if VE is not enabled for that title

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

Jdforrester-WMF added a commit: rEVEDc6ea5d44a7a7: Allow parsefragment even if VE is not enabled for that title.Apr 2 2015, 5:00 PM

DannyH moved this task from Untriaged to Resolved on the Collaboration-Team-Triage board.Apr 3 2015, 12:28 AM

EBernhardson mentioned this in rOMWCd5bc66d4920e: Revert "Enable VisualEditor for Flow posts on Beta Labs".Apr 21 2015, 12:41 AM

Jdforrester-WMF edited a custom field.Aug 5 2015, 1:20 AM

Jdforrester-WMF moved this task from To Triage to TR1: Releases on the VisualEditor board.Aug 10 2015, 3:39 PM

Jdforrester-WMF removed a project: Patch-For-Review.Sep 1 2015, 10:53 PM

Jdforrester-WMF added a project: VisualEditor 2014/15 Q4 blockers.Sep 10 2015, 7:14 PM

• Mattflaschen-WMF added a commit: rEVEDe194a6fd36f5: Allow parsefragment even if VE is not enabled for that title.Apr 28 2016, 1:35 AM

Way for VE to run on Flow without changing wgVisualEditorNamespacesClosed, ResolvedPublic0 Estimated Story PointsActions