Page MenuHomePhabricator

[PageTools] Set up feature flag for page tools menu
Closed, ResolvedPublic2 Estimated Story Points

Description

NOTE: No blockers for this work

Background

Prior to beginning the work on the page tools menu, we would like to set up a feature flag to ensure that the feature is not seen in production

Acceptance criteria

  • Set up feature flag for page tools menu and corresponding url parameter

Event Timeline

LGoto set the point value for this task to 2.

Change 802197 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Add article tools menu feature flag

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

Change 802200 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Define feature classes with FeatureManager

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

bwang removed bwang as the assignee of this task.Jun 1 2022, 9:26 PM
bwang moved this task from Doing to Code Review on the Web-Team-Backlog (Kanbanana-FY-2021-22) board.
bwang subscribed.

I'm moving this to blocked for now, as I'm realizing we now have several feature flags that have implications on layout and it's becoming really difficult to reason with either of them. Adding another at this point seems risky, so I'd suggest before adding this one we remove at least one of the following:
$wgVectorTableOfContents, $wgVectorGrid, $wgVectorTitleAboveTabs, $wgVectorStickyHeader

Change 802200 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Define feature classes with FeatureManager

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

Looking to merge this sometime next week.

We should capture https://gerrit.wikimedia.org/r/802200 in a separate ticket as that seems like super useful functionality but is not as high a priority.

After chatting to Olga we decided to put this back in the backlog for now until the grid and table of contents work is completed.

Change 802197 abandoned by Bernard Wang:

[mediawiki/skins/Vector@master] Add article tools menu feature flag

Reason:

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

Change 802200 abandoned by Jdlrobson:

[mediawiki/skins/Vector@master] Define feature classes with FeatureManager

Reason:

Done in https://gerrit.wikimedia.org/r/c/mediawiki/skins/Vector/+/829041

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

Jdlrobson renamed this task from Set up feature flag for article tools menu to [ArticleTools] Set up feature flag for article tools menu.Sep 15 2022, 4:11 PM
Jdlrobson updated the task description. (Show Details)
Jdlrobson lowered the priority of this task from High to Low.Sep 19 2022, 5:29 PM

(these have previously been estimated)

Change 802197 restored by Jdlrobson:

[mediawiki/skins/Vector@master] Add article tools menu feature flag

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

Change 802197 abandoned by Bernard Wang:

[mediawiki/skins/Vector@master] Add article tools menu feature flag

Reason:

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

Change 836262 had a related patch set uploaded (by Bernard Wang; author: Bernard Wang):

[mediawiki/skins/Vector@master] Add Article Tools feature flag

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

bwang removed bwang as the assignee of this task.Sep 28 2022, 5:56 PM

Change 836262 had a related patch set uploaded (by Jdlrobson; author: Bernard Wang):

[mediawiki/skins/Vector@master] Add Article Tools feature flag

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

Change 836262 merged by jenkins-bot:

[mediawiki/skins/Vector@master] Add Article Tools feature flag

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

alexhollender_WMF renamed this task from [ArticleTools] Set up feature flag for article tools menu to [PageTools] Set up feature flag for article tools menu.Oct 25 2022, 7:20 PM
alexhollender_WMF renamed this task from [PageTools] Set up feature flag for article tools menu to [PageTools] Set up feature flag for page tools menu.Oct 25 2022, 7:25 PM
alexhollender_WMF updated the task description. (Show Details)