Maniphest T320666

[M] Ensure that QuickView mobile functionality isn't released until Milestone 3/mobile MVP is complete
Closed, ResolvedPublic
Actions

Description

Similar to how we are keeping QuickView desktop behind a config until Milestone 2 is complete and we have an MVP, we want to do the same for mobile functionality. This ticket is to determine and implement a method to ensure that mobile functionality is either hardcoded or behind a config variable so that as we release desktop functionality, we can still hide mobile functionality until it's tested and the MVP is ready.

n.b. implementation should also include a query string parameter to override the feature flag for testing purposes.

Details

	Subject	Repo	Branch	Lines +/-
	Add temporary toggle for disabling on mobile	mediawiki/extensions/SearchVue	master	+24 -2

Customize query in gerrit

Related Objects

Mentioned In: T328136: Switch mobile Search Preview config to on
T326957: [M] Remove configuration setting that turns off/on mobile Search Preview
rESVUea3e015ec228: Add temporary toggle for disabling on mobile

Event Timeline

CBogen created this task.Oct 12 2022, 5:41 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 12 2022, 5:41 PM

CBogen edited projects, added Structured-Data-Backlog (Current Work); removed Structured-Data-Backlog.Oct 17 2022, 4:32 PM

CBogen moved this task from Incoming to Ready for Estimation on the Structured-Data-Backlog (Current Work) board.

MarkTraceur renamed this task from Ensure that QuickView mobile functionality isn't released until Milestone 3/mobile MVP is complete to [M] Ensure that QuickView mobile functionality isn't released until Milestone 3/mobile MVP is complete.Oct 19 2022, 5:48 PM

MarkTraceur updated the task description. (Show Details)

MarkTraceur moved this task from Ready for Estimation to Ready for Development on the Structured-Data-Backlog (Current Work) board.Oct 19 2022, 5:59 PM

Also tagging milestone 2. Since there will be some overlap in work, we ought to make sure that anything mobile remains invisible until it is fully ready.

Change 848175 had a related patch set uploaded (by Matthias Mullie; author: Matthias Mullie):

[mediawiki/extensions/SearchVue@master] Add temporary toggle for disabling on mobile

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

gerritbot added a project: Patch-For-Review.Oct 24 2022, 7:57 AM

Above patch will ensure SearchVue remains disabled on mobile except:

when explicitly enabled globally, via $wgQuickViewEnableMobile
when &quickViewEnableMobile=1 is detected in the query string (which will allow spot-testing even in environments where it's globally disabled)