Subject index

[gunnarvelle]

Add indexing of subjects

[gunnarvelle]

Dersom målet på sikt er å kunne søke på alle noder, bør du sikkert også ha med contexts for å støtte alle plasseringer en node kan ha i struktur. Dersom kun fag skal vises, er det du har nok.

[gunnarvelle]

Usikker på om det er problematisk, men for alle andre søkeresultat så peiker url på api-url, men det returneres liste med context som har url som er korrekt frontendurl. Burde kanskje gjort det likt her.

[gunnarvelle]

Er det meininga at det skal være NodeHitDTO?

"results": [
    {
      "id": "urn:subject:d1fe9d0a-a54d-49db-a4c2-fd5463a7c9e7",
      "title": "Tverrfaglige temaer",
      "url": "http://localhost:30017/f/tverrfaglige-temaer/daaf4e2dd8b0",
      "typename": "NodeHitDTO"
    }, ...
]```

[jnatten]

Er det meininga at det skal være NodeHitDTO?

"results": [
    {
      "id": "urn:subject:d1fe9d0a-a54d-49db-a4c2-fd5463a7c9e7",
      "title": "Tverrfaglige temaer",
      "url": "http://localhost:30017/f/tverrfaglige-temaer/daaf4e2dd8b0",
      "typename": "NodeHitDTO"
    }, ...
]```

Har du frontpage-api kjørende med data?

[gunnarvelle]

Bedre!:

{
      "id": "urn:subject:d1fe9d0a-a54d-49db-a4c2-fd5463a7c9e7",
      "title": "Tverrfaglige temaer",
      "url": "http://localhost:30017/f/tverrfaglige-temaer/daaf4e2dd8b0",
      "subjectPage": {
        "id": 239,
        "name": "Tverrfaglige tema",
        "metaDescription": {
          "metaDescription": "Folkehelse og livsmestring, demokrati og medborgerskap, bærekraftig utvikling",
          "language": "nb"
        }
      },
      "typename": "NodeHitDTO"
    }

[gunnarvelle]

Er denne egentlig klar for review?

[jnatten]

Er denne egentlig klar for review?

Jeg kom på at jeg hadde lagt igjen en todo på suggestions, så jeg burde sikkert se mer på det før vi prøver å merge den.
Alt annet er jo klart, men jeg vet ikke helt hva som er galt med den så kan jo kanskje bli uforutsette endringer 🤔

[jnatten]

Tror denne er klar for review nå.

Sluttet å bruke dynamisk mapping rundtomkring med grunnlag i at vi mister litt kontroll på hva som kan søkes på og ikke; da det fort er basert på dataen som er indeksert.

Feks problemet som oppstod her var at vi ikke kunne bruke suggestions på multi-index søket i search-api når vi fikk inn node indexen siden suggestionen gjør at hele søket på node sharden feiler siden feltet "content.nb" ikke fantes.
Selvom vi la inn dynamisk mapping på det så vil ikke feltet finnes før det faktisk kommer inn data.

Ser for meg at det er lett å skyte seg selv i foten med senere dersom vi bruker et sjeldent språk i en av index'ene for eksempel.

Siden vi fjerner dynamisk mapping så får vi litt begrensninger på hvor mange språk vi kan støtte samtidig med denne modellen, så la inn et konfigurasjonsflagg for å bestemme hvilke fag som støttes.

Vurderte å endre modellen til å bruke nested dokumenter for språkfelter å fremdeles kunne støtte alle språkene samtidig, men følte det kanskje var et unødvendig ytelse-hit om vi kan leve med å støtte litt færre språk samtidig. Syns den løsningen hadde vært "cleanere" kodemessig, men mtp usikker ytelse så blir nok dette "tryggest" 🤓

[jnatten]

Disse er de som finnes i ed pr nå, men tar i mot forslag på hvilke vi trenger.
Jo færre jo bedre egentlig da 🤓

[gunnarvelle]

Kan merge denne no regner eg med.