Change 519799 had a related patch set uploaded (by DannyS712; owner: DannyS712):
[mediawiki/extensions/MassMessage@master] Allow editing spamlist descriptions using the API

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

Would Platform Engineering be willing to review the patch? MassMessage is used on the WMF cluster

hey @DannyS712, what's the priority for this? We're a little behind on external code reviews at the moment but trying to catch up.

In T226929#5444407, @WDoranWMF wrote:

hey @DannyS712, what's the priority for this? We're a little behind on external code reviews at the moment but trying to catch up.

Its a nice feature to have, but not too urgent. Its blocking allowing Twinkle to interact properly with mass message lists (see https://github.com/azatoth/twinkle/issues/680). I added multiple tests, so hopefully code review shouldn't take too long

In T226929#5444412, @WDoranWMF wrote:

Ok cool, you can see our backlog on CPT Clinic Duty External Code Reviews. I'll put it as Normal for now which likely means next week. If the priority jumps up or that doesn't work let me know and I can do my best to reprioritise.

Thanks, that's no problem. I know that I already have 4 other tasks in the external code review column, so no pressure.

@WDoranWMF any updates?

Ah apologies, it ended up in the wrong column, I'll move it over.

Ah apologies, it ended up in the wrong column, I'll move it over.

@DannyS712 my apologies, I think this was my mistake. CPT is not actually the owner of MassMessage, according to the Maintainers List, it's actually Editing-team, I'm tagging them here. If this is incorrect we can also take a look.

@ Editing-team just want to make sure this is on your radar. The patch has been pending for a month now. Core platform team has a dedicated column for tracking external code reviews needed - is there something specific I need to do with this task for editing team to review?

In T226929#5476271, @WDoranWMF wrote:

@DannyS712 my apologies, I think this was my mistake. CPT is not actually the owner of MassMessage, according to the Maintainers List, it's actually Editing-team, I'm tagging them here. If this is incorrect we can also take a look.

Would you be willing to take a look, given the lack of response from Editing-team?

Change 519799 merged by jenkins-bot:
[mediawiki/extensions/MassMessage@master] Allow editing spamlist descriptions using the API

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