Page MenuHomePhabricator

Security review for the DiscussionTools extension
Closed, DuplicatePublic

Event Timeline

Yes, I'm creating this task structurally, but no idea when the team will raise the request through the usual channels. That's for @JTannerWMF/@ppelberg.

Yes, I'm creating this task structurally, but no idea when the team will raise the request through the usual channels. That's for @JTannerWMF/@ppelberg.

Thank you, @Jdforrester-WMF 🙏🏼.

I think this task is talking about similar things as T242134 and so I'm presumptively merging the two.

Yes, I'm creating this task structurally, but no idea when the team will raise the request through the usual channels. That's for @JTannerWMF/@ppelberg.

Thank you, @Jdforrester-WMF 🙏🏼.

I think this task is talking about similar things as T242134 and so I'm presumptively merging the two.

Then you should put that task into the hierarchy that it's blocking. (Also, you should generally merge into the older task…)

Then you should put that task into the hierarchy that it's blocking. (Also, you should generally merge into the older task…)

My mistake. For now, I've added T242134 as a child of this ticket.

@ppelberg - So this task and T242134 are not ostensibly the same thing? If they are, we can merge them. Only reason I'm asking is because there doesn't seem to be anything actionable on this task, so at the very least I would probably untag Application Security Reviews.

In the future we will just update this one as it is older to @Jdforrester-WMF point

Hi @sbassett , I've merged the tasks.

Ok, great!

In the future we will just update this one as it is older to @Jdforrester-WMF point

So this task (older one) appears to be the one that was closed as a dup while T242134 (newer one) remains open...