-
Notifications
You must be signed in to change notification settings - Fork 44
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
Migrate the Elastica Grid driver from Sylius/Sylius PR #1
Comments
Yes no problem :) |
Perfect :) |
Hi @Niiko when #6 will be merged you can open pr with your driver and it would be great if you will apply some changes from comments here Sylius/Sylius#5756. Also you don't have to register those services i will combine your commits with #7 and make it works 🎉 . |
Hi @Arminek ok, will do that :) thanks for support |
Hi @Arminek, due to sylius issue closed, i'm a little bit confused on how to contribute. If i'm correct, driver can still be included and has to be for grid usage no ? Otherwise we can't use it for. Just have to make it compliant with the new engine and match method, right ? |
[ElasticSearch] Rename & remove ui & bump dsl to 5.0
@Niiko
Sylius/Sylius#5756
We should migrate the driver here to kickstart the development of this bundle. Are you willing to submit a PR here or we somehow get your commits here?
The text was updated successfully, but these errors were encountered: