search

pdphilip / laravel-elasticsearch

Laravel Elasticsearch: An Elasticsearch implementation of Laravel's Eloquent ORM
MIT License
86 stars 16 forks source link

3.10.x #26

Closed Polokij closed 5 months ago

pdphilip commented 4 months ago

Hey @Polokij - interesting PR though I assume you closed it to work on it some more.

Just a comment, composer deps for all Laravel versions is where I started at but turned out to be untenable. The package directly inherits and extends Laravel's database layer (Query, Eloquent, Relations, etc) which tend to evolve over each Major Laravel version - hence the package needs to be adapted for each and thus has a branch pegged by Laravel version.

Then, what are your thoughts on opting for opensearch-php in favour of the ES's official client? Keen to hear

pdphilip commented 4 months ago

@Polokij - I've gone deep into OpenSearch and have rebuilt this package to integrate with it directly.

It's in dev-main for now and you can install it with composer require pdphilip/opensearch:dev-main

See: https://github.com/pdphilip/laravel-opensearch

Important to note the new structure for database.php and the .env vars. And the namespacing change to PDPhilip\OpenSearch\...

It should be available very soon as I've cleared all the tests. Feel free to move aspects of this PR that side.