Constructor dependency injector avoids Laravel Fortify to work out of the box in Tenancy for Laravel package

laravel / fortify

Backend controllers and scaffolding for Laravel authentication.

https://laravel.com/docs/fortify

MIT License

1.61k stars 294 forks source link

Constructor dependency injector avoids Laravel Fortify to work out of the box in Tenancy for Laravel package #163

Closed acacha closed 3 years ago

acacha commented 3 years ago

According to:

https://tenancyforlaravel.com/docs/v3/early-identification

Using constructor DI avoids tenancy route middleware to works. AFAIK the proposed changes like using method injection instead of constructor will solve the issue.

I can write a Pull Request if your think the change is viable. Feel free to close this issue for any reason.

driesvints commented 3 years ago

If you want to change anything just PR it, thanks

acacha commented 3 years ago

PR created at https://github.com/laravel/fortify/pull/169 . Sorry for the multiple PR with errors.