laravel-scout-elasticsearch icon indicating copy to clipboard operation
laravel-scout-elasticsearch copied to clipboard

Published 20 hours ago • verified publisher icon matchish

Poll: Lazymap for Mixedsearch

Open matchish opened this issue 2 years ago • 4 comments

Vote the post if you need this feature

matchish avatar May 26 '22 09:05 matchish

Please add this. Laravel Nova has a scout integration, but it relies on ElasticSearchEngine->lazyMap(...) being implemented.

m-lotze avatar Jul 21 '22 15:07 m-lotze

https://github.com/matchish/laravel-scout-elasticsearch/pull/205 @hkulekci added lazymap already. And I think for Nova have to be enough. Original scout don't have MixedSearch functionality so I don't think Nova requires lazyMap for MixedSearch. But I didn't work with Nova so can be wrong.

matchish avatar Jul 22 '22 13:07 matchish

My bad. Did not see it was implemented in v6. We are unfortunately stuck on elasticsearch version 7 for now. any chance you can add it in v5 also?

m-lotze avatar Jul 25 '22 07:07 m-lotze

PR for 6x should be compatible with 5x. So if someone will create 5x branch and cherry pick https://github.com/matchish/laravel-scout-elasticsearch/pull/205 there it have to work

matchish avatar Jul 25 '22 08:07 matchish