-
Notifications
You must be signed in to change notification settings - Fork 60
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
hot feature - DynamicWidget Support. #1014
Comments
@bidoubiwa |
Is this project dead? |
Hey @pixelkoduje, Not sure why it does not work, there a lot of things we need to check compatibility with. I'll look at it as soon as possible. Unfortunaly I'm going on holiday. When I come back I will definitely try to fix that :) Unless someone contributes of course |
It was the same in typesense - it didn't work either. You can look at their code if you want. @bidoubiwa |
Has anything moved regarding dynamicWidgets? |
I repeat the question. This is a much needed function that is bugged and does not work, and there are no contraindications for it to work. @bidoubiwa |
Hey @pixelkoduje! As stated above, I was on holiday :) We are currently in pre-release, so I cannot guarantee to be able to look at this compatibility issue asap. I'll try to get into it when I have the time to do so. Feel free to investigate as well as what is required to make it work if you want the issue to be resolved faster 🔥 |
You wrote that after the holidays it will move ... I sent everything I can to help. Same problem was with typesense. |
It seems to me that things that should work and do not work - i.e. bugs - should be fixed first, before new features. |
You are free to contribute and fix the issue :) |
I will be very grateful to @bidoubiwa if this can be taken care of. As I wrote, I am introducing a project for many countries, with millions of e-commerce products. And this will be a lot of advertising for meilisearch as well. And I think the community will like it. After all, this is the thing that doesn't work - and meilisearch is a cheaper alternative to algolia after all. |
@bidoubiwa Thanks a lot for your answer. I see it's not worth it. apply here. Bye |
As per the suggestion, take inspiration from these two commits on typesense side to implement it here: |
All you have to do is create a project and it will be passed, right?
wt., 22 sie 2023, 14:00 użytkownik cvermand ***@***.***>
napisał:
… As per the suggestion, take inspiration from these two commits on
typesense side to implement it here:
- ***@***.***
<typesense/typesense-instantsearch-adapter@1016e7a>
- ***@***.***
<typesense/typesense-instantsearch-adapter@3d10297>
—
Reply to this email directly, view it on GitHub
<#1014 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYMU3BVVGELO6ESY5666Z6LXWSNM7ANCNFSM6AAAAAAVKMCMVQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi, |
Hello @Saadchr Any contribution is welcome to make it compatible 😊 |
Hello, may I know why DynamicWidget is not working? There are all the things that make it work. All "facets" are returned correctly - but DynamicWidget doesn't see them. I suspect it's a minor thing - and it will make meilisearch skyrocket in terms of usability. After all, it was created, rather, for larger projects - and each larger project has hundreds of categories and hundreds of "facets". I created the same thread in typesense and they created an implementation in a moment. I'm creating a huge ecommerce project with hundreds of brands and millions of products. And meilisearch I see it does better with facets- if there is a large amount and returns immediately.
Info on how they did it so you can see.
typesense/typesense-instantsearch-adapter#150
The text was updated successfully, but these errors were encountered: