Strange to see this added here and then removed in a later diff in the stack

I felt like this would make reviewing easier, as those two diffs have quite different purposes.
This diff implements logic for login, following what we already have in the code for registration. This is code we know works, we are just implementing it for an action that was recently added.
The next diff refactors this logic. The refactor is due to us having rethought some general app design, and is more likely to be questioned and possibly changed drastically or abandoned.
So I wanted to have this first diff that can be safely landed before we agree on the second diff. This is also partially because the second diff was not in the initial scope of my goal.

Okay, fair enough!