it looks to me like the previous code would also update model.signedIn with false; is that an intentional change? If so it should be mentioned in the commit message why it was changed (ideally it's a separate change, really)