Open pulkitjalan opened 9 years ago
Hi @pulkitjalan - sorry for the delay. This looks great, and something that I'd love to get into this repo, and there are a lot of great other changes in here as well. Let me pull this down and give it a try over the next day or so, and then I'll probably set up a new branch for you to make the pull request to.
Thats great, looking forward to it. Just a quick note, I added facets to the response collection before I knew it was deprecated. I'll will be removing that soon. On 2 Jan 2015 04:30, "Adam Fairholm" notifications@github.com wrote:
Hi @pulkitjalan https://github.com/pulkitjalan - sorry for the delay. This looks great, and something that I'd love to get into this repo, and there are a lot of great other changes in here as well. Let me pull this down and give it a try over the next day or so, and then I'll probably set up a new branch for you to make the pull request to.
— Reply to this email directly or view it on GitHub https://github.com/adamfairholm/Elasticquent/issues/13#issuecomment-68508429 .
Hi @adamfairholm Please use this new branch instead of the other one. I have removed facets from it.
Hi @pulkitjalan - Sorry for the delay - I am going to take a look at this this weekend. Thanks!
Hi @adamfairholm have you had a chance to look at this?
Hi @adamfairholm,
I don't know if you would be up for merging this in, but I thought there were some things missing from this great package. Eager loading relationships and also indexing using chunking, more specifically the
reindex
andaddAllToIndex
not chunking. I have made these and some other changes on a fork. If you get some time please have a look and let me know if its something you would be happy to merge in and I will make a pull request.https://github.com/clowdy/Elasticquent/tree/relations