Ryan Bilesky
Ryan Bilesky
I was actually kinda surprised myself this didn't include this myself. I threw together the functionality on my own. 1. I went into the source grabbed the register view code,...
> I'd personally would just ditch the auth integration and leave the user info data for middleware. I get the appeal but it in most cases it will end up...
> To me, this PR doesn't really hold its weight. You're changing several public methods in foundational classes of the framework to save developers a few keystrokes when chaining `->all()`....
> I feel the same as @jasonmccreary . Changing all those methods just to save `->all()` is not worth it. Idk I can see it go either way. I'd argue...
> How did you decide which methods to update and which to not? For example, why is `BelongsToMany@createOrFirst` not updated? @taylorotwell That and any other methods I did not include...
I know it can be tempting to just implement fortify or go the fortify route of just directly integrating something simple like TOTP and calling it good. I for one...