Lag lovdata lenker

[selbekk]

Denne endringen gjør at vi lenker til den relevante siden av Folketrygdloven på Lovdata hver gang vi viser en lovreferanse i løsningen vår.

Målet er at det skal være raskere for saksbehandlere å finne frem til riktig sted i lovteksten.

Jeg la også den nye koden inn under v2/gui/shared, siden den brukes på tvers av hele løsningen vår.

[selbekk]

Skrev tester i storybook for å sentralisere alt på ett sted. Mitt første forsøk på storybook-tester, så åpen for innspill!

[josstn]

Syns koden såg mycket bra ut 👍

Det eg lure litt på er dog om det kanskje er litt risikabelt å gjere ei tolkning og endring av tekst som blir lagt inn i "slot" slik som dette ut frå tolkning. Om det kan føre til uheldige overaskelser sidan eg ville tru folk ikkje forventer at ein slik tekst potensielt blir forandra av koden.

Meir konkret prøvde eg å legge inn følgande storytest:

export const ParagrafUtenMellomromEtter: Story = {
  args: {
    children: '§9-1',
  },
  play: async ({ canvasElement, step }) => {
    const { linkEls, root} = elemsfinder(canvasElement);
    await step('§ uten mellomrom etter blir korrekt', async () => {
      await expect(linkEls()).toHaveLength(1);
      const [firstLink] = linkEls();
      await expect(firstLink).toHaveTextContent('9-1');
      await expect(firstLink).toHaveAttribute('href', 'https://lovdata.no/lov/1997-02-28-19/§9-1');
      await expect(root()).toHaveTextContent('§9-1')
    });
  },
};

Dette førte til at link vart printa ut, men ikkje § prefiks.

Testa og ein der eg putta inn '§§9-1, 9-2 og §9-3', då vart berre 9-1 og 9-3 printa ut, så potensielt forsvant ein viktig lovreferanse då.

Det eg tenkje er at for ein slik funksjonalitet bør feilmodus vere slik at ein er sikker på at tekst blir printa ut slik den kom inn, og at linkgenerering heller feiler, slik at ein unngår at tekst som er sendt inn forsvinner.

[josstn]

Dette funke fint, men med siste versjon av Storybook vart dette den anbefalte måten slik eg har forstått det:

const meta = {
  title: 'gui/shared/lovreferanse/Lovreferanse.tsx',
  component: Lovreferanse,
} satisfies Meta<typeof Lovreferanse>;
export default meta

Med denne koden får ein betre typescript sjekk av args.

Fjerna og withMaxWidth decorator, ser ikkje at den har noko nytteverdi for denne story.

[selbekk]

Ah topp! Tok bare utgangspunkt i noe jeg fant fra før av, men skal oppdatere i vei!

[josstn]

Med siste Storybook er og dette anbefalt måte å definere Story typen på:
type Story = StoryObj<typeof meta>;

[selbekk]

@josstn gode innspill.

Vi har heldigvis full kontroll på teksten som dukker opp via denne filen i k9-sak. Det kommer en PR som normaliserer måten vi skriver ting på der også, så det oppleves likere på tvers av vilkårene.

Når det er sagt, så skal jeg skrive noen tester som validerer at vi aldri ender opp med å fjerne noen tekst.

[selbekk]

Relevant PR i k9-sak som nok burde ut først: https://github.com/navikt/k9-sak/pull/10935