Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Monotonize aggregating page-content by sequencing page-elements' standard_html #281

Merged
merged 1 commit into from
May 30, 2024

Conversation

drfho
Copy link
Contributor

@drfho drfho commented May 29, 2024

Ref:

  1. 936b091
  2. Reindexing Context's Page Container on Change/Insert/Delete #278

Result:

1 Now page content is aggreated the same way for all cases (fullindex, change, insert).
2. In case of indexed attributes (e.g. standard_html) are not filled with expected text, delete (misconfigured) search-adapters and start full reindex.

unibe_reindex

URLs:

  1. CONFIG https://edit.cmstest2.unibe.ch/unibe/content/zcatalog_adapter/manage_main
  2. TEST https://edit.cmstest2.unibe.ch/unibe/portal/fak_vetmedizin/a_dept_dkv/inst_kleintier/content/e18883/e750745/e750831/manage_main?lang=ger
  3. INDEX https://idcmsosearchint01.id-sys.unibe.ch/app/dev_tools#/console

ad 3: Snippets

GET /unibe/_search
{
  "query": {
    "match_phrase": {
      "uid": {
      "query": "\"uid:c8b129ae-2f1d-4ad3-83ad-0c7606893394\""
      }
      
    }
  }
}

GET /unibe/_search
{
  "query": {
    "bool": {
      "must": [
        {
          "simple_query_string": {
            "query": "inst_kleintier",
            "fields": ["home_id"]
          }
        },
        {
          "simple_query_string": {
            "query": "ger",
            "fields": ["lang"]
          }
        },
        {
          "simple_query_string": {
            "query": "e18868*",
            "fields": ["zmsid"]
          }
        }
      ]
    }
  }
}

@drfho drfho requested a review from OscarMeier May 29, 2024 22:31
@drfho drfho merged commit 7827358 into unibe/opensearch May 30, 2024
13 checks passed
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