-
Notifications
You must be signed in to change notification settings - Fork 3
Reduce number of queries executed in the WikiUserEmailChecker service #1043
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
a6d32f4 to
3cd57d9
Compare
outdooracorn
left a comment
There was a problem hiding this 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.
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
|
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 |
Co-authored-by: Ollie <43674967+outdooracorn@users.noreply.github.com>
f9f399b to
a57277f
Compare
outdooracorn
left a comment
There was a problem hiding this 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! ![]()
fix mistake from merge
Reduce number of queries executed in the
WikiUserEmailCheckerservice in order to improve performance.Bug: T415017