Open k1ng440 opened 8 years ago
@HipsterJazzbo can you accept this pull request please? https://github.com/AuraEQ/laravel-multi-tenant/pull/77
Yeah this will need a few minutes to bring it back down to 4.2 compatibility. I won't have time today, but if someone does a PR I'll merge it
please accept this too #78
I totally get not wanting to support older laravel versions and appreciate that you are still accepting pull requests.
thanks!
Someone let me know if it works now on 4.2 with #78 merged, and I'll tag a version
In regards to Laravel 5.1, i'm not ready yet to upgrade to 5.2, so i'll have to stick with this one for now, but an issue has occured.
I'm not entirely sure what's happening. But for some reason it's not injecting it properly on the models.
I have it configured the usual way, and added the addTenant(); before doing anything database related, but the queries are not updated to include the tenant-scope. Not sure if it's related to the fix in #78 ?
Alright, issue found.
The problem is the naming of the config file as far as i can see.
When doing the publish method, it's calling the file "laravel-multi-tenant.php" whereas the code is looking for a file named "tenant.php".
This should resolve my before-mentioned issue :)
Hello @HipsterJazzbo, can you review and accept #79 please?
Sorry, done.
Hello once again @HipsterJazzbo, please review and accept https://github.com/HipsterJazzbo/laravel-multi-tenant/pull/80
thanks for creating this awesome lib.
Hello @HipsterJazzbo, i am using 5.1 LTS and this is the error i am getting now