Informasjonsforvaltning / fdk-portal

Apache License 2.0
3 stars 3 forks source link

BUG: Løsningen støtter ikke at dokumentasjonskrav kan være et datasett #1620

Open KurtStian opened 1 year ago

KurtStian commented 1 year ago

🐛 Bug rapport

Nåværende oppførsel

Kun dokumentasjonskrav (cpsv:hasInput) som relaterer til klassen cv:Evidence støttes.

Hvis man legger til en URL til et fiktivt datasett og legger inn en klasse dcat:Dataset med dct:title og dct:description i høstefilen vil informasjon denne tittelen og beskrivelsen vises under tjenesteresultat. Dette vil også vises som datasett (ikon og blå ramme), men den vil ikke være klikkbar (med ikon som viser at lenken går ut av FDK).

Hvis man kun legger til URL til et fiktivt datasett så vises ikke noe.

Hvis man legger til URI til et datasett som finnes i FDK på staging så vises ikke noe.

image image

Forventet oppførsel

Relasjon til datasett, både de som finnes i FDK og utenfor FDK, skal vises. Det skal fremgå av GUI at det er datasett og de skal være klikkbar. URI som peker ut av FDK skal merkes med gjeldende ikon for dette. Hvis filen inneholder også innholder klasse dcat:Datasett med dct:title og dct:description for URI som peker ut av FDK skal dette vises, hvis ikke vises kun URI.

Hvordan reprodusere?

Se tjenesteresultat for tjeneste: https://staging.fellesdatakatalog.digdir.no/public-services/309d00a5-1887-314b-a609-a639905c3ab9

Forslag/Løsning [Valgfritt]

Sjekkliste (Sjekk av etter issue er laget)


terjesyl commented 1 year ago

Parseren støtter differensiering av Evidence og Dataset, som vist i JSON-responsen. Må sjekke om frontend skiller riktig mellom disse: bilde

~URI-er i RDF-grafen som peker til ressurser utenfor grafen blir merket med rdfType "UNKNOWN" i JSON-responsen. Kan bruke det som flagg for å markere ekstern lenke.~ (kan droppe UNKNOWN, se kommentar under) bilde

Hva tenker du @NilsOveTen ?

Jeg kjenner ikke til hvordan lenkingen (intern, ekstern) i frontend gjøres, kan du undersøke det @hegeaal ?

terjesyl commented 1 year ago

Så vidt jeg kan se må vi:

Krever endringer i frontend. Hvilke? Vi kan også fjerne "UNNOWN"-typen i parseren, siden vi antar at alt som ikke er Evidence er instans av Dataset.

Hva tror dere, @NilsOveTen @hegeaal ?

Hva tenker du om URL-er som ikke er reelle eller brutte, @KurtStian ? Vi vil jo inntrykk av at det er fungerende lenker, men tror ikke vi kan ta for gitt at alle URI-er vi høster er det.

KurtStian commented 1 year ago

@terjesyl Samme utfordring er det ved begrep relatert til datasett. Der valgte vi å foreløpig ikke ha noe sjekk om URL er reelle (f.eks. sjekke om den gir response 200), og heller avvente å se om dette blir et problem. Mener at vi foreløpig kan gjøre det samme her.

Men på sikt burde vi lage en funksjon for å sjekke at det er fungerende lenker som kan gjenbrukes over alt i løsningen der det brukes/kan brukes lenker. Vi har jo snakket om en tilsvarende funksjon for å kunne avdekke der det er lenkeråte slik at vi kan manuelt/automatisk følge opp mot eier av beskrivelsen - som et kvalitetsforbedringstiltak.

KurtStian commented 1 year ago

Feilen er relatert til testing av https://github.com/Informasjonsforvaltning/behov/issues/650