Skip to content

Fix for indexer_update_all_views cron job#8

Closed
RaphaelBronsveld wants to merge 1 commit into
comwrap:masterfrom
RaphaelBronsveld:master
Closed

Fix for indexer_update_all_views cron job#8
RaphaelBronsveld wants to merge 1 commit into
comwrap:masterfrom
RaphaelBronsveld:master

Conversation

@RaphaelBronsveld
Copy link
Copy Markdown

@RaphaelBronsveld RaphaelBronsveld commented Aug 16, 2019

Hi there,

We have been using this module for a few days now and we noticed one of our cron jobs failed (indexer_update_all_views). Every change you'd make in the backend (product, category), would never be visible on the frontend (without running the indexers manually). Pretty heavy bug!

It was because of the following error;
Missing required argument $data of Comwrap\ElasticsuiteBlog\Model\Post\Indexer\Fulltext.

We are using Magento version: 2.3.2

Reproducing:
Local environment with this elasticsuite + this module + magefan blog module
php bin/magento cron:run first time to schedule
php bin/magento cron:run second time to run the scheduled jobs

Thanks in advance!
Gr, Raphael

@nuzil nuzil requested a review from Neos2007 August 28, 2019 08:03
@nuzil
Copy link
Copy Markdown
Collaborator

nuzil commented Aug 28, 2019

Hi @RaphaelBronsveld thanks a lot for this PR, as before we did not supported 2.3.2, this was an issue.
We updated version yesterday with 2.3.2 support, so your PR is included there.

@nuzil nuzil closed this Aug 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants