Page MenuHomePhabricator

Rename ConfigurationLoader service to NewcomerTasks-specific name
Closed, ResolvedPublic

Description

We are working to move many configuration to on-wiki pages, see the parent task. This will require a service that will handle config-related issues. However, we currently have a service called GrowthExperimentsConfigurationLoader and GrowthExperimentsConfigurationValidator that imply they handle GrowthExperiments-wide configuration, while they're actually newcomertasks-specific.

Let's rename those services to NewcomerTasks-specific name, so it's less confusing for when we actually merge patches that will work on the parent task.

Event Timeline

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

Change 663055 had a related patch set uploaded (by Urbanecm; owner: Urbanecm):
[mediawiki/extensions/GrowthExperiments@master] Rename ConfigurationLoader service to NewcomerTasks-specific name

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

Change 663055 merged by jenkins-bot:
[mediawiki/extensions/GrowthExperiments@master] Rename ConfigurationLoader service to NewcomerTasks-specific name

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

Urbanecm_WMF added a subscriber: Etonkovidova.

@Etonkovidova Hi, moving this to QA, as the only patch got merged.