GrowthExperiments: Make use of Suggested edits config in Community configuration
Open, HighPublic
Actions

Description

As part of T360471: Growth features: Migrate Suggested edits from Special:EditGrowthConfig to Community configuration 2.0, we created Special:CommunityConfiguration/GrowthSuggestedEdits as a form for managing Suggested edits configuration. The form works (makes it possible to define how Suggested edits work like), but the configured data is not used at all, as PageConfigurationLoader is always using MediaWiki:NewcomerTasks.json and loading it directly.

Within this task, we need to make Special:CommunityConfiguration/GrowthSuggestedEdits actually change how GrowthExperiments behaves. We will do this by replacing the custom loading logic with a callout to CommunityConfiguration.

Details

	Subject	Repo	Branch	Lines +/-
	Switch to CommunityConfiguration for SuggestedEdits config	mediawiki/extensions/GrowthExperiments	master	+500 -304

Customize query in gerrit

Related Objects

Mentioned In: T359185: Community configuration: Define behaviour for external titles and wikipage-based configuration store
Mentioned Here: T360471: Growth features: Migrate Suggested edits from Special:EditGrowthConfig to Community configuration 2.0

Event Timeline

Urbanecm_WMF created this task.Mon, May 13, 8:42 AM

Restricted Application added a project: Growth-Team. · View Herald TranscriptMon, May 13, 8:42 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Change #1030961 had a related patch set uploaded (by Urbanecm; author: Urbanecm):

[mediawiki/extensions/GrowthExperiments@master] Switch to CommunityConfiguration for SuggestedEdits config

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

gerritbot added a project: Patch-For-Review.Mon, May 13, 1:54 PM

Urbanecm_WMF mentioned this in T359185: Community configuration: Define behaviour for external titles and wikipage-based configuration store.Mon, May 13, 1:55 PM

Sgs moved this task from Backlog to Migrate Special:EditGrowthConfig on the CommunityConfiguration board.Mon, May 13, 5:19 PM

Urbanecm_WMF claimed this task.Mon, May 13, 5:20 PM

Urbanecm_WMF triaged this task as High priority.

Urbanecm_WMF edited projects, added Growth-Team (Sprint 13 (Growth Team)); removed Growth-Team.

Urbanecm_WMF moved this task from Incoming to Code Review on the Growth-Team (Sprint 13 (Growth Team)) board.

Urbanecm_WMF edited projects, added Growth-Team (Sprint 14 (Growth Team)); removed Growth-Team (Sprint 13 (Growth Team)).Tue, May 14, 4:13 PM

DMburugu moved this task from Incoming to QA on the Growth-Team (Sprint 14 (Growth Team)) board.Tue, May 14, 5:08 PM

Sgs moved this task from QA to Code Review on the Growth-Team (Sprint 14 (Growth Team)) board.Tue, May 14, 5:12 PM

Sgs moved this task from Code Review to QA on the Growth-Team (Sprint 14 (Growth Team)) board.Thu, May 16, 12:11 PM

Does the merged change account for the missing properties in T360471#9804253?

Change #1030961 merged by jenkins-bot: