Reimport: clearing objects in the initial index if the temporary index has zero objects#186
Reimport: clearing objects in the initial index if the temporary index has zero objects#186borealsmith wants to merge 9 commits intoalgolia:masterfrom
Conversation
… newly imported objects
nunomaduro
left a comment
There was a problem hiding this comment.
@chloelbn will jump on this.
|
@chloelbn Can you review this one? |
|
Hello @borealsmith, Thanks for your PR! I’ve reviewed and the code seems fine, great work! Would you mind adding some tests along your feature, to ensure everything is working as intended? You can take as an example the test for the Reimport Command (https://github.com/algolia/scout-extended/blob/master/tests/Features/ReimportCommandTest.php) and add in this file a test case with 0 object to import. |
|
Hi @chloelbn, Thank you very much for the review and feedback. Of course, no problem, my pleasure to add the test case you mention. Best Regards, |
|
Hi, @nunomaduro @chloelbn |
|
Hi, |
|
can we merge this? |
No description provided.