search

staudenmeir / eloquent-has-many-deep

Laravel Eloquent HasManyThrough relationships with unlimited levels
MIT License
2.67k stars 157 forks source link

Push to laravel / framework? #177

Closed sidewaysglance closed 1 year ago

sidewaysglance commented 2 years ago

This package is so useful, thank you!

It is performant as it makes proper use of the database, and is much more versatile than standard eloquent relations.

Why not push to https://github.com/laravel/framework?

staudenmeir commented 1 year ago

Thanks @sidewaysglance. With all the additional functionality, the package's code and documentation have grown quite a bit over time and so I think it's better to keep it as a separate package.

robman70 commented 1 year ago

Oh, damn! I use the Query Log all the time to look for errors, this time (I don't know why), I didn't. In fact, an SQL error did not occur but the query sequence was wrong and a constraint I had imposed on the relations prevented the update. Thanks so much for the help!

staudenmeir commented 1 year ago

@robman70 Wrong ticket ;-)

robman70 commented 1 year ago

Oops... sorry! I was reading this ticket and replied to this one instead of the one I opened

decadence commented 1 year ago

I second this (and the same request for belongs-to-through package). I think every Laravel user would be happy to have those in core.

Anyway great work @staudenmeir. Thanks for creating and maintaining these packages!