Page MenuHomePhabricator

[structured mentor list] Replace action=growthsetmentorstatus with action=growthmanagementorlist
Closed, ResolvedPublic

Description

Thanks to T264343 / structured mentor list, we now have an "umbrella API endpoint" to manage information about mentors. Once the structured mentor list gets deployed (T310905), we should replace action=growthsetmentorstatus with action=growthmanagementorlist.

Checklist:

Event Timeline

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

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

[mediawiki/extensions/GrowthExperiments@master] Mentor dashboard: Deprecate ApiSetMentorStatus

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

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

[mediawiki/extensions/GrowthExperiments@master] [i18n] Add missing documentation for action=growthmanagementorlist API

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

Sgs changed the task status from Open to In Progress.Jul 7 2023, 12:21 PM
Sgs moved this task from Incoming to In Progress on the Growth-Team (Sprint 0 (Growth Team)) board.

Change 936091 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] [i18n] Add missing documentation for action=growthmanagementorlist API

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

Change 936090 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Mentor dashboard: Deprecate ApiSetMentorStatus

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

Added a draft to the Tech news.

Summary for User-notice: The growthsetmentorstatus API will be deprecated in the wmf.18 train in favour of the growthmanagementorlist API, which can do everything growthsetmentorstatus can, plus a bunch of other things. It is likely the growthsetmentorstatus API will be fully removed fairly soon (we'll check usages before removing anything).

Urbanecm_WMF changed the task status from In Progress to Stalled.Jul 13 2023, 10:16 AM

Now waiting on the announcement, wmf.18 train deployment and the waiting period. Starting Aug 1, we should check for usages and decide whether the endpoint can be safely removed.

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

[mediawiki/extensions/GrowthExperiments@master] Remove ApiSetMentorStatus

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

In [2]: df = spark.run("SELECT COUNT(*) FROM event.mediawiki_api_request WHERE year=2023 AND (month=8 OR (month=7 AND day > 20)) AND params['action'] = 'growthsetmentorstatus'"); df
[...]
Out[2]: 
   count(1)
0         0
In [3]: df = spark.run("SELECT COUNT(*) FROM event.mediawiki_api_request WHERE year=2023 AND (month=8 OR (month=7 AND day > 20)) AND params['action'] = 'growthmanagementorlist'"); df
[...]

Seems like the old API is not used by external users at all (which matches expectations). Let's remove it!

Urbanecm_WMF changed the task status from Stalled to Open.Aug 2 2023, 4:29 PM
Urbanecm_WMF updated the task description. (Show Details)

Change 944956 merged by jenkins-bot:

[mediawiki/extensions/GrowthExperiments@master] Remove ApiSetMentorStatus

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

Urbanecm_WMF updated the task description. (Show Details)