search

navikt / diagnosekoder

MIT License
0 stars 0 forks source link

Forslag: Flytte søke-koden ut i et eget lib #50

Open karl-run opened 9 months ago

karl-run commented 9 months ago

Jeg foreslår at man kan lage en ny package som heter noe alà @navikt/diagnosekoder-sok hvor all den søkekoden legges. Sånn at "grunn-pakken" @navikt/diagnosekoder er kun selve kodene. :smile:

josstn commented 9 months ago

Eg er ikkje nødvendigvis i mot dette altså, men kunne tenkje meg litt meir bakgrunnsinfo for forslaget, kvifor tenkjer du at det er ønskelig?

Kan og nemne at eg i utgangspunktet ønskte å publisere dette "på ein meir moderne måte", som ein pakke med separate ESM moduler utan noko rollup, slik at konsumentane kunne importere berre dei filene som var behov for (import {DiagnosekodeSearcher} from "@navikt/diagnosekoder/DiagnosekodeSearcher" istadenfor import {DiagnosekodeSearcher} from "@navikt/diagnosekoder" Det fungerte i utgangspunktet, men feila når eg skulle teste med jsdom i konsumenten (sjå https://github.com/navikt/diagnosekoder/commit/f92357518b81035ec5ebc0eea9c0f1afc192aa8a)

Tenkte ein slik publiseringsmåte kunne vore det beste, sidan ein då aldri ville vere borti koden ein ikkje trengte, men det ville framleis kunne ligge i samme prosjekt, så ein unngjekk ekstra vedlikeholdskostnad for fleire prosjekt/pakker.

karl-run commented 9 months ago

Det var hovedsaklig fordi jeg som consumer av denne libben er kun interessert i kodene, så når jeg skal importere kodene og får dette som forslag i editoren:

image

Så tenker jeg at denne libben var mer enn jeg ba om. Det havner jo seff ikke i bundlen så det er jo ikke et "problem" i den forstand. :)

josstn commented 9 months ago

Ok, du hadde kanskje synest det var ok viss pakken var publisert som separate ESM moduler under @navikt/diagnosekoder og då, slik eg først prøvde?

Viss eg skal bruke tid på dette er eg meir gira på å prøve å få det til igjen i alle fall, sidan eg trur det kunne vere ein nyttig framgangsmåte fleire stader 🙂

Er mindre gira på å lage eit prosjekt til som må haldast oppdatert for noko så lite. Men viss du har lyst å ta det på deg er det ok for meg 🙂

karl-run commented 9 months ago

Ja det funker det. :smile: Har ikke så veldig sterke formeninger om det. :sweat_smile: