Skip to content
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

laravel-scout-extended integration #123

Open
1 of 2 tasks
curquiza opened this issue May 12, 2021 · 12 comments
Open
1 of 2 tasks

laravel-scout-extended integration #123

curquiza opened this issue May 12, 2021 · 12 comments
Labels

Comments

@curquiza
Copy link
Member

curquiza commented May 12, 2021

Context

At first, we decided to build an integration for Laravel Scout by staying completely compliant/conformed with the original laravel/scout package. See meilisearch-laravel-scout.

The goal? To integrate this repository into the official laravel/scout package (as Algolia already is) by showing that people use it. This is now done! 🎉

But, despite this laravel-scout integration, we noticed some of the contributors would like a laravel-scout-extended integration, I mean with more friendly methods to handle settings in this repository, as this PR suggests. We refused this kind of PRs in meilisearch-laravel-scout since it would make our integration non-compliant with the official laravel/scout.

Decision with the community

According to this issue, and after the merge of the MeiliSearch support into Laravel/Scout, we have decided to:

@curquiza curquiza added enhancement New feature or request ready to implement labels May 12, 2021
@Braunson
Copy link

Braunson commented Sep 5, 2021

@curquiza Any progress or need any help developing the repo laravel-scout-extended. I was going to fork the Algolia one and work on modifying/stripping it to work with Meilisearch myself.

@curquiza
Copy link
Member Author

curquiza commented Sep 6, 2021

Hello @Braunson!

Thanks for this suggestion! We are not working currently on it, so any help is really interested!! 😄

@curquiza curquiza added new integration For the new integrations and removed enhancement New feature or request labels Oct 10, 2021
@curquiza
Copy link
Member Author

curquiza commented Oct 22, 2021

People want the sort feature, see: laravel/scout#537

@LiamKarlMitchell
Copy link

Aggregates, multiple models from same index is something I found my self reaching for when using with Lighthouse graphql and its paginator.

Working around it by using meilisearch-php and querying directly for now.
Some kind of a union or merge with primary key and a "type of model" field, but keeping the order/pagination.

https://github.com/algolia/scout-extended/blob/master/src/Searchable/Aggregator.php
https://www.algolia.com/doc/framework-integration/laravel/advanced-use-cases/multiple-models-in-one-index/?client=php

@brunoocasali
Copy link
Member

Thanks for your comment @LiamKarlMitchell :)

@brunoocasali
Copy link
Member

Hi all!

As you probably know, Laravel Scout released a new version v9.5.0 which introduced a new ability to set Meilisearch's index settings 🎉.

So, my question is, for everyone watching this issue, do you think a laravel-scout-extended is still something worthy to be created? And which kind of feature do you expect to be present on it if your answer were "yes".

Thanks a lot!

@smknstd
Copy link

smknstd commented Dec 12, 2022

Hello, this is great news !

I liked the artisan command scout:status which gives me lots of information as an overview:

  • a list of all indexes
  • model used
  • synchronisation status
  • record count

https://www.algolia.com/doc/framework-integration/laravel/production/get-status-overview/?client=php

Screenshot 2022-12-12 at 13-41-51 Get status overview Laravel Algolia

@MrThePlague
Copy link

So, my question is, for everyone watching this issue, do you think a laravel-scout-extended is still something worthy to be created? And which kind of feature do you expect to be present on it if your answer were "yes".

I'd love to see support of aggregates in order to implement a multi-model/global search.

@curquiza
Copy link
Member Author

Update: looks like the multi-search is missing to users: laravel/scout#745

@brunoocasali
Copy link
Member

This is something it could be tackled by the laravel-scout-extended since it will not be supported: laravel/scout#730 (comment)

@MrThePlague
Copy link

Hi folks, is this actively being worked on? 😍

@brunoocasali
Copy link
Member

Hi, @MrThePlague hope you're doing well!

Unfortunately, even though the community highly requests it, we haven't progressed on this topic yet. It would be nice to have it, but we need more time to work on it. We are a small team that maintains more than 30 tools, SDKs and integrations, so it is pretty hard to make the time 😅

It would be nice if someone from the community could step in and start the project. We can provide help and feedback when needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants