Skip to content

Add include tag for main content

Sarah German requested to merge elastic-data-tags into main

What does this MR do and why?

Adjusts markup to go back to using data attributes rather than the custom field for the main content area.

The custom field did not have the same sort of formatting as body_content does (e.g, snippets did not have any spaces between paragraphs like we have with body_content). The default body_content field must have some sort of extra processing behind it.

Let's see if using an include rather than excludes works any better. We seem to be getting different behavior than what the docs suggest (https://www.elastic.co/guide/en/enterprise-search/current/crawler-content.html#crawler-content-inclusion-and-exclusion).

How to set up and validate locally

n/a, need to test on production after a crawl runs.

Merge request acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading