Skip to content

Commit

Permalink
Merge pull request #1865 from navikt/FixAntallAnsatteBuckets
Browse files Browse the repository at this point in the history
Fix antall ansatte buckets
  • Loading branch information
ebelegu authored Sep 17, 2024
2 parents 1db12c7 + 741b7af commit 1318eca
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 9 deletions.
2 changes: 2 additions & 0 deletions src/api/enhetsregisteretApi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,6 +43,7 @@ const Underenhet = z
forretningsadresse: Adresse,
hjemmeside: z.string(),
overordnetEnhet: z.string(),
harRegistrertAntallAnsatte: z.boolean(),
antallAnsatte: z.number(),
beliggenhetsadresse: Adresse,
institusjonellSektorkode: Kode,
Expand All @@ -56,6 +57,7 @@ const Underenhet = z
forretningsadresse: true,
hjemmeside: true,
overordnetEnhet: true,
harRegistrertAntallAnsatte: true,
antallAnsatte: true,
beliggenhetsadresse: true,
institusjonellSektorkode: true,
Expand Down
25 changes: 16 additions & 9 deletions src/utils/funksjonerForAmplitudeLogging.ts
Original file line number Diff line number Diff line change
Expand Up @@ -30,15 +30,19 @@ export const loggSidevisning = (pathname: string) => {
});
};

export const finnBucketForAntall = (antall: number | undefined) => {
if (antall === undefined) {
return undefined;
}
export const finnBucketForAntall = (
harRegistrertAntallAnsatte: boolean | undefined,
antall: number | undefined
) => {
if (harRegistrertAntallAnsatte === undefined) return;

//Hvis harRegistrertAntallAnsatte er false er det 0 ansatte
if (!harRegistrertAntallAnsatte) return '0';

//Hvis harRegistrertAntallAnsatte er true og antall er undefined er det 1-4 ansatte
if (antall === undefined) return '1-4';

switch (true) {
case antall === 0:
return '0';
case antall < 5:
return '1-4';
case antall < 20:
return '5-19';
case antall < 50:
Expand Down Expand Up @@ -91,7 +95,10 @@ export const useLoggBedriftValgtOgTilganger = (org: OrganisasjonInfo | undefined

if (underenhet !== undefined) {
virksomhetsinfo.sektor = finnSektorNavn(underenhet);
virksomhetsinfo.antallAnsatte = finnBucketForAntall(underenhet.antallAnsatte);
virksomhetsinfo.antallAnsatte = finnBucketForAntall(
underenhet.harRegistrertAntallAnsatte,
underenhet.antallAnsatte
);
}

amplitude.logEvent('virksomhet-valgt', virksomhetsinfo);
Expand Down

0 comments on commit 1318eca

Please sign in to comment.