Skip to content

Conversation

@evgeni
Copy link
Contributor

@evgeni evgeni commented Jan 9, 2026

this was enabled in the past and then accidentally disabled

Fixes: 1f1975f

this was enabled in the past and then accidentally disabled

Fixes: 1f1975f
Copy link
Contributor

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume you tested this?

@evgeni
Copy link
Contributor Author

evgeni commented Jan 10, 2026

Of course! ;)

(I did, really!)

@evgeni
Copy link
Contributor Author

evgeni commented Jan 10, 2026

the commit that disabled that says "- Commented out content indexing in plugin code but added comprehensive content indexing in JavaScript", but this doesn't seem to work -- as there is no "content" in the generated json, only when you enable the plugin code to add content, then json has it and JS can search

@felixfontein
Copy link
Contributor

In the same code in 1f1975f there's also a lot of usage of data[key].content, so it doesn't look right that content isn't put into the JSON.

@felixfontein felixfontein merged commit 68f89ad into getnikola:master Jan 10, 2026
7 checks passed
@felixfontein
Copy link
Contributor

Thanks for fixing this @evgeni!

@evgeni evgeni deleted the flexsearch-content branch January 11, 2026 07:58
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