-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Support laravel 5.3 #925
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support laravel 5.3 #925
Conversation
New Laravel queries always return collection. this is a backward compatible enhancement as Collections can be used as Arrays!
Hopes this gets merged, there is 1 test failing though , but is failing on master anyways |
* | ||
* @var boolean | ||
*/ | ||
protected $use_collection; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be camel case.
Hello , |
@alsemany True, but not related to this PR. according to official README, Supported relations are :
|
Any idea when this will be merged? I started a project in 5.3 just to find out that this repo is not compatible yet :/ |
@jenssegers any chance to merge this PR? |
I'll merge this, but I can only tag a new release once we have fixed some of the tests though. |
Remove shouldUseCollections function which checks Laravel version > 5.3, it was introduced to support version 5.3 in mongodb#925
Remove shouldUseCollections function which checks Laravel version > 5.3, it was introduced to support version 5.3 in #925
Initial support laravel 5.3
Remove shouldUseCollections function which checks Laravel version > 5.3, it was introduced to support version 5.3 in mongodb#925
New Laravel queries always return collection. this is a backward compatible enhancement as Collections can be used as Arrays!