877369 - search: result summary should use get_summary()

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: Search, defect, P1)

Description

[Luke Crouch [:groovecoder]]

Index get_summary() output into a 'summary' field in ES.

Show this summary in the results - highlighted if possible.

Comment 1

[[github robot]]

Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/71d1b00cd8c7f6a068264090342fd303e735dd1f
fix bug 877369 - use seoSummary for result display

https://github.com/mozilla/kuma/commit/ce4c28500cbb8195f1c1140a5df7ca240f732555
Merge pull request #1225 from jezdez/search-summary-877369

fix bug 877369 - use seoSummary for result display

Comment 2

[Luke Crouch [:groovecoder]]

The code is written to do this, but after our change to get_summary in https://github.com/mozilla/kuma/pull/1262/files it doesn't seem to stick? Needs investigation and fixing.

Comment 3

[Jannis Leidel [:jezdez]]

Hm, I'm not able to reproduce that it doesn't stick on on developer-dev.allizom.org. When I search for "many" via https://developer-dev.allizom.org/en-US/search?q=many I get the summary of that page. I've before switched the SEO summary from a different part of the text and the change was reflected in an updated search index entry.

Can you elaborate what didn't work for you?

Comment 4

[Luke Crouch [:groovecoder]]

Ah, I'm still confused about excerpt vs. summary displaying in the search results. It's working correctly. We might get similar feedback from users though, we'll see.