Page MenuHomePhabricator
Create Task
Maniphest T94282

Way for VE to run on Flow without changing wgVisualEditorNamespaces
Closed, ResolvedPublic0 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

Commits
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

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
gerritbot added a subscriber: gerritbot.Apr 1 2015, 11:55 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
• Mattflaschen-WMF updated the task description. (Show Details)Apr 2 2015, 12:10 AM
• Mattflaschen-WMF set Security to None.
gerritbot added a comment.Apr 2 2015, 12:11 AM

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

gerritbot added a comment.Apr 2 2015, 12:42 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

gerritbot added a comment.Apr 2 2015, 4:55 PM

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

gerritbot added a comment.Apr 2 2015, 4:58 PM

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

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

gerritbot added a comment.Apr 2 2015, 4:58 PM

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 edited a custom field.Aug 5 2015, 1:20 AM
Content licensed under Creative Commons Attribution-ShareAlike 3.0 (CC-BY-SA) unless otherwise noted; code licensed under GNU General Public License (GPL) or other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL