Added a comand for checking if user email exist in api and MW user tables#1031
Added a comand for checking if user email exist in api and MW user tables#1031rosalieper merged 5 commits intomainfrom
Conversation
tarrow
left a comment
tarrow
left a comment
There was a problem hiding this comment.
Looks like a good direction to start in. Definitely a bit different from the raw string matching in the ticket but seems a fine idea to me. I like the test. I guess you're also working on one for Wiki data too?
One comment I added for myself to worry about because I have some vague memory about these connections being a bit odd when we set them like this but I need to do some research
|
|
||
| $manager = app()->db; | ||
| $manager->purge('mw'); | ||
| $mwConn = $manager->connection('mw'); |
There was a problem hiding this comment.
For Tom to look up:
- do we also need to undo this?
AndrewKostka
left a comment
AndrewKostka
left a comment
There was a problem hiding this comment.
I tried it out and it works for me locally. I added a few suggestions for ways we could reduce the number of queries if we run into any performance issues. However, the benefits only kick in for larger email lists. I don't suspect we'll use this feature that heavily, so I'd ignore these suggestions for now.
4 participants
Bug: T410856