search

djangonaut-space / wagtail-indymeet

Djangonaut Space site
MIT License
13 stars 9 forks source link

Don't update wagtail reference index for survey responses #350

Closed tim-schilling closed 3 months ago

tim-schilling commented 3 months ago

Follow-up to #349

tim-schilling commented 3 months ago

We may need to find a better way to do this in our app. I'm also not sure how it all works, but this is one option available to us.

sarahboyce commented 3 months ago

I think this problem goes away when we upgrade to wagtail 5+ https://docs.wagtail.org/en/v5.1.3/advanced_topics/reference_index.html

tim-schilling commented 3 months ago

Good find.

tim-schilling commented 3 months ago

Solved in #360

tim-schilling commented 3 months ago

@sarahboyce just read the docs link. I'm not sure I agree. Can you highlight the part that says that this is solved?

sarahboyce commented 3 months ago

"When introduced in Wagtail 4.1, the Reference Index recorded references in all application models by default. Wagtail 5.0 reduced the scope of the default models to those specifically linked with Wagtail." https://docs.wagtail.org/en/v5.1.3/advanced_topics/reference_index.html#configuration

tim-schilling commented 3 months ago

Thank you. I'll need to rerun this test without the ignore prop to confirm then.