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

Fix inventory management Fix #35 #39

Merged
merged 3 commits into from
Jun 6, 2016

Conversation

romainruaud
Copy link
Collaborator

No description provided.

@romainruaud romainruaud added this to the v 2.1 milestone Jun 3, 2016
@afoucret
Copy link
Contributor

afoucret commented Jun 3, 2016

Not huge fan of injecting ResourceConnection into src/module-elasticsuite-catalog/Plugin/Indexer/AbstractIndexerPlugin.php and handling SQL here.

Would be better if we inject a resource model and working with it.

@romainruaud
Copy link
Collaborator Author

I have updated the code.

@romainruaud romainruaud added the bug label Jun 3, 2016
@afoucret afoucret merged commit 8c48cc6 into Smile-SA:master Jun 6, 2016
afoucret pushed a commit that referenced this pull request Jun 6, 2016
@romainruaud romainruaud deleted the fix_inventory-management branch October 27, 2016 13:27
botisSmile pushed a commit to botisSmile/elasticsuite that referenced this pull request Sep 8, 2021
…n-autocomplete-requests

Adding autocomplete queries to explain #ESP-140
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants