Skip to content

Conversation

@rosalieper
Copy link
Contributor

@rosalieper rosalieper commented Jan 22, 2026

Reduce number of queries executed in the WikiUserEmailChecker service in order to improve performance.

Bug: T415017

Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Left some minor nitpick comments.

rosalieper and others added 2 commits January 22, 2026 17:00
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
@outdooracorn
Copy link
Member

The PR title doesn't seem quite right to me. How about "Refactor WikiUserEmailChecker service"?

It would also be good to say why you made this refactor. What are the advantages?

And a Bug: git trailer.

@rosalieper rosalieper changed the title Refactor gdpr script Refactor WikiUserEmailChecker service Jan 22, 2026
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
@outdooracorn outdooracorn changed the title Refactor WikiUserEmailChecker service Reduce number of queries executed in the WikiUserEmailChecker service Jan 23, 2026
Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great stuff, happy for this to be merged! :shipit:

Base automatically changed from T415017 to main January 23, 2026 13:28
@deer-wmde deer-wmde merged commit 0b8596f into main Jan 23, 2026
5 checks passed
@deer-wmde deer-wmde deleted the refactorGDPR-script branch January 23, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants