mattporritt / moodle-search_elastic

An Elasticsearch engine plugin for Moodle's Global Search

https://moodle.org/plugins/search_elastic

GNU General Public License v3.0

16 stars 13 forks source link

general feedback #8

Closed brendanheywood closed 7 years ago

brendanheywood commented 7 years ago

[ ] $item->index->status >= 300 would be future proof
[ ] add protection from network failure around json_decode and $client->post ( $docurl, $payload )->getBody

mattporritt commented 7 years ago

Hi Brendan, I've just updated master with the latest code (I haven't updated plugin directory yet). It's a fairly large refactor and also now supports batch document processing. The batch processing won't land until Moodle 3.4 (https://tracker.moodle.org/browse/MDL-58654). The plugin is backwards compatible until then, but won't batch process.

It should also be more resilient to the states you've mentioned

Matt P

ghost commented 7 years ago

mattporritt / moodle-search_elastic

general feedback #8

11 Typo fix PR