Skip to content

WIP Update kitconcept.solr with FHNW improvements#314

Open
reekitconcept wants to merge 7 commits into
mainfrom
update-kitconcept-solr-with-fhnw-improvements
Open

WIP Update kitconcept.solr with FHNW improvements#314
reekitconcept wants to merge 7 commits into
mainfrom
update-kitconcept-solr-with-fhnw-improvements

Conversation

@reekitconcept
Copy link
Copy Markdown
Member

@reekitconcept reekitconcept commented Jan 28, 2026

Summary

  • Update kitconcept.solr
  • Add Solr highlighting fields configuration
  • Add upgrade step v20260128001

Test plan

  • Run upgrade step on existing installation
  • Verify highlighting fields appear in search results
  • Test search functionality with new configuration with Solr disabled
  • Test search functionality with new configuration with Solr enabled

@reekitconcept reekitconcept requested review from a team as code owners January 28, 2026 15:11
@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch from 96544c2 to 24da6a0 Compare January 28, 2026 15:13
@reekitconcept reekitconcept changed the title Add Solr highlighting fields configuration Update kitconcept.solr with FHNW improvements Jan 28, 2026
@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch 3 times, most recently from a02aba8 to 375d3a0 Compare January 28, 2026 15:20
@reekitconcept reekitconcept changed the title Update kitconcept.solr with FHNW improvements WIP Update kitconcept.solr with FHNW improvements Jan 28, 2026
@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch 5 times, most recently from 7636cb9 to 30a545a Compare January 29, 2026 15:14
@reekitconcept reekitconcept requested a review from a team as a code owner January 29, 2026 15:14
@reekitconcept
Copy link
Copy Markdown
Member Author

reekitconcept commented Jan 29, 2026

Highlighting improvements from the new branch now works out-of-the-box.

Previously:

  • Title highlighting was not supported
  • Description highlighting showed body text snippets instead of actual description content

Now:

  • Title highlighting works correctly
  • Description highlighting shows proper description content

Both are returned in dedicated fields (highlighting_title, highlighting_description)

Firefox_Screenshot_2026-01-29T15-16-48 093Z

The other improvements regarding suggestions, "Did you mean to..." are not used as we continue using the default suggest workflow from the frontend. Fhnw has its customized frontend to use these. We might at some point want to use it too, but likely only in a next major version with a redesign.

@reekitconcept reekitconcept requested review from davisagli and removed request for a team January 29, 2026 15:24
@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch from 30a545a to b0560ce Compare January 29, 2026 15:31
Comment thread backend/pyproject.toml Outdated
"pas.plugins.oidc==2.0.0",
"collective-solr==10.0.0",
"kitconcept.solr==2.0.0a9",
"kitconcept.solr @ git+https://github.com/kitconcept/kitconcept.solr.git@fhnw-features-e5ac96d#subdirectory=backend",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Needs to be updated with an actual release before we merge this.

Comment thread frontend/package.json
"prettier": "prettier --check 'packages/**/src/**/*.{js,jsx,ts,tsx}' '!packages/volto-solr/**'",
"prettier:fix": "prettier --write 'packages/**/src/**/*.{js,jsx,ts,tsx}' '!packages/volto-solr/**'",
"stylelint": "stylelint 'packages/**/src/**/*.{css,scss,less}' '!packages/volto-solr/**' --allow-empty-input",
"stylelint:fix": "stylelint 'packages/**/src/**/*.{css,scss,less}' '!packages/volto-solr/**' --fix --allow-empty-input",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@reekitconcept We should add this permanently in frontend/.eslintignore instead of here. I'm not sure if we have something similar for prettier and stylelint.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I agree - we use the same from fhnw for eslint.

Re prettier and stylelint: I'm not sure what's best.

@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch from 98fe9e3 to de76933 Compare February 4, 2026 17:17
@reekitconcept reekitconcept force-pushed the update-kitconcept-solr-with-fhnw-improvements branch from de76933 to e4f0374 Compare February 4, 2026 17:23
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