Merge branch 'main' into context-refaktorering

# Conflicts: # src/Pages/Hovedside/InfoBokser.tsx # src/Pages/Saksoversikt/Saksfilter/Saksfilter.tsx # src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/Virksomhetsmeny.tsx
navikt · Dec 27, 2024 · 6a9f614 · 6a9f614
2 parents 9eb562f + 566b83b
commit 6a9f614
Show file tree

Hide file tree

Showing 12 changed files with 78 additions and 77 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -8,8 +8,8 @@
     "@faker-js/faker": "8.4.1",
     "@grafana/faro-web-sdk": "1.3.6",
     "@navikt/aksel-icons": "^5.17.2",
-    "@navikt/arbeidsgiver-notifikasjon-widget": "7.2.0",
-    "@navikt/bedriftsmeny": "7.0.4",
+    "@navikt/arbeidsgiver-notifikasjon-widget": "7.2.1",
+    "@navikt/bedriftsmeny": "7.0.5",
     "@navikt/ds-css": "^6.15.0",
     "@navikt/ds-icons": "3.4.3",
     "@navikt/ds-react": "^6.15.0",

diff --git a/src/Pages/Hovedside/InfoBokser.tsx b/src/Pages/Hovedside/InfoBokser.tsx
@@ -17,18 +17,6 @@ type InfoboksProps = {
 };
 
 const infobokser: Array<InfoboksProps> = [
-    {
-        id: 'uxsignals',
-        visFra: new Date('2024-11-06T00:00:00+02:00'),
-        visTil: new Date('2024-12-31T10:00:00+02:00'),
-        Component: () => {
-            const { valgtOrganisasjon } = useOrganisasjonsDetaljerContext();
-            if (!valgtOrganisasjon || !valgtOrganisasjon.altinntilgang.inntektsmelding) {
-                return null;
-            }
-            return <UXSignals dataUxsignalsEmbed="panel-mwwvs1lq0" />;
-        },
-    },
     {
         id: 'ia-bookings',
         visFra: new Date('2023-11-13T10:00:00+02:00'),
@@ -109,6 +97,9 @@ export const InfoBokser = () => {
     );
 };
 
+/**
+ * fjernet bruk av denne inntil vi har kontroll på konsekvenser iht ny ekomlovgivning
+ */
 const UXSignals = ({ dataUxsignalsEmbed }: { dataUxsignalsEmbed: string }) => {
     useEffect(() => {
         const script = document.createElement('script');

diff --git a/src/Pages/Saksoversikt/Saksfilter/Saksfilter.tsx b/src/Pages/Saksoversikt/Saksfilter/Saksfilter.tsx
@@ -50,7 +50,7 @@ const KollapsHvisMobil: FC<KollapsHvisMobilProps> = ({
 }: KollapsHvisMobilProps) => {
     if (width < 730) {
         return (
-            <Ekspanderbartpanel tittel="Filtrering" ikon={<FilterIkon />}>
+            <Ekspanderbartpanel tittel="Filtrering" ikon={<FilterIkon aria-hidden="true" />}>
                 {children}
             </Ekspanderbartpanel>
         );
@@ -127,6 +127,7 @@ const InntektsmeldingGruppe = (
         navn.includes('Inntektsmelding')
     );
 
+
     if (inntektsmeldingAlleValgtAvBruker) {
         valgteInntektsmeldingtyper = ['Inntektsmelding_gruppe'];
     } else if (andreInntektsmeldingerValgt) {
@@ -312,12 +313,11 @@ export const Saksfilter = ({
                     </CheckboxGroup>
                 )}
 
-                <CheckboxGroup legend="Virksomheter" className="saksfilter_virksomhetsmeny">
-                    <Virksomhetsmeny
-                        valgteEnheter={valgteVirksomheter}
-                        setValgteEnheter={setValgteVirksomheter}
-                    />
-                </CheckboxGroup>
+                <Virksomhetsmeny
+                    valgteEnheter={valgteVirksomheter}
+                    setValgteEnheter={setValgteVirksomheter}
+                />
+
                 <OpprettInntektsmelding />
             </div>
         </KollapsHvisMobil>
@@ -352,8 +352,12 @@ const OpprettInntektsmelding = () => {
                     paddingBottom: '32px',
                 }}
             >
-                <Label children="Opprett inntektsmelding manuelt" />
+                <Label
+                    htmlFor="opprett-inntektsmelding-lenke-id"
+                    children="Opprett inntektsmelding manuelt"
+                />
                 <LenkeMedLogging
+                    id="opprett-inntektsmelding-lenke-id"
                     loggLenketekst={'Opprett inntektsmelding manuelt'}
                     href={opprettInntektsmeldingURL}
                 >

diff --git a/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/HovedenhetCheckbox.css b/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/HovedenhetCheckbox.css
diff --git a/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/HovedenhetCheckbox.tsx b/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/HovedenhetCheckbox.tsx
@@ -1,6 +1,5 @@
 import { BodyShort, Checkbox } from '@navikt/ds-react';
 import React from 'react';
-import './HovedenhetCheckbox.css';
 import { Set } from 'immutable';
 import { Organisasjon } from '../../../../altinn/organisasjon';
 import { amplitudeFilterKlikk } from '../Saksfilter';
@@ -14,22 +13,19 @@ export const HovedenhetCheckbox = ({ hovedenhet, valgteOrgnr }: HovedenhetCheckb
     const valgt = valgteOrgnr.has(hovedenhet.OrganizationNumber);
 
     return (
-        <div role="button" aria-expanded={valgt}>
-            <div className="hovedenhet">
-                <Checkbox
-                    size="small"
-                    value={hovedenhet.OrganizationNumber}
-                    id={`${hovedenhet.OrganizationNumber}_Virksomhetsmeny_checkbox`}
-                    key={`${hovedenhet.OrganizationNumber}_Virksomhetsmeny_list_key`}
-                    style={{ display: 'flex', alignItems: 'center' }}
-                    description={`Org.nr. ${hovedenhet.OrganizationNumber}`}
-                    onClick={(e) => amplitudeFilterKlikk('organisasjon', 'hovedenhet', e.target)}
-                >
-                    <BodyShort size="medium" as="span">
-                        {hovedenhet.Name}
-                    </BodyShort>
-                </Checkbox>
-            </div>
-        </div>
+        <Checkbox
+            aria-expanded={valgt}
+            size="small"
+            value={hovedenhet.OrganizationNumber}
+            id={`${hovedenhet.OrganizationNumber}_Virksomhetsmeny_checkbox`}
+            key={`${hovedenhet.OrganizationNumber}_Virksomhetsmeny_list_key`}
+            style={{ display: 'flex', alignItems: 'center' }}
+            description={`Org.nr. ${hovedenhet.OrganizationNumber}`}
+            onClick={(e) => amplitudeFilterKlikk('organisasjon', 'hovedenhet', e.target)}
+        >
+            <BodyShort size="medium" as="span">
+                {hovedenhet.Name}
+            </BodyShort>
+        </Checkbox>
     );
 };
diff --git a/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/UnderenhetCheckboks.tsx b/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/UnderenhetCheckboks.tsx
@@ -11,11 +11,6 @@ type UnderenhetCheckboksProps = {
 
 export const UnderenhetCheckboks = ({ underenhet, valgteOrgnr }: UnderenhetCheckboksProps) => {
     return (
-        <div
-            className="sak_virksomhetsmeny_underenhet"
-            role="menuitemcheckbox"
-            aria-checked={valgteOrgnr.has(underenhet.OrganizationNumber)}
-        >
             <Checkbox
                 value={underenhet.OrganizationNumber}
                 size="small"
@@ -29,6 +24,6 @@ export const UnderenhetCheckboks = ({ underenhet, valgteOrgnr }: UnderenhetCheck
                     {underenhet.Name}{' '}
                 </BodyShort>
             </Checkbox>
-        </div>
+
     );
 };
diff --git a/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/Virksomhetsmeny.tsx b/src/Pages/Saksoversikt/Saksfilter/Virksomhetsmeny/Virksomhetsmeny.tsx
@@ -1,5 +1,5 @@
 import React, { ReactNode, useMemo, useState } from 'react';
-import { CheckboxGroup, Search } from '@navikt/ds-react';
+import { CheckboxGroup, Label, Search } from '@navikt/ds-react';
 import './Virksomhetsmeny.css';
 import { UnderenhetCheckboks } from './UnderenhetCheckboks';
 import { HovedenhetCheckbox } from './HovedenhetCheckbox';
@@ -116,8 +116,10 @@ export const Virksomhetsmeny = ({
 
     return (
         <>
+            <Label htmlFor="virksomheter_checkbox_group_id"> Virksomheter </Label>
             <Søkeboks onChange={onSearchChange} />
             <CheckboxGroup
+                id="virksomheter_checkbox_group_id"
                 legend="Velg virksomheter"
                 hideLegend
                 value={valgteEnheter.toArray()}
@@ -153,7 +155,7 @@ export const Virksomhetsmeny = ({
                                             }
 
                                             return [
-                                                <li key={underenhet.OrganizationNumber}>
+                                                <li key={underenhet.OrganizationNumber} className="sak_virksomhetsmeny_underenhet">
                                                     <UnderenhetCheckboks
                                                         valgteOrgnr={valgteEnheter}
                                                         underenhet={underenhet}

diff --git a/src/mocks/brukerApi/alleNotifikasjoner.ts b/src/mocks/brukerApi/alleNotifikasjoner.ts
@@ -2,6 +2,24 @@ import { KalenderavtaleTilstand, OppgaveTilstand } from '../../api/graphql-types
 import { beskjed, dateInPast, kalenderavtale, oppgave } from './helpers';
 
 export const alleNotifikasjoner = [
+    beskjed({
+        tekst: "Oppfølgingsplan til godkjenning",
+        sakTittel: "",
+        klikketPaa: false,
+        lenke: "https://demo.ekstern.dev.nav.no/syk/oppfolgingsplaner/arbeidsgiver/123"
+    }),
+    oppgave({
+        tekst: 'Du er innkalt til dialogmøte - vi trenger svaret ditt',
+        klikketPaa: false,
+        tilstand: OppgaveTilstand.Ny,
+        lenke: 'https://demo.ekstern.dev.nav.no/syk/dialogmoter/arbeidsgiver/123',
+    }),
+    oppgave({
+        tekst: 'Dialogmøtet med NAV er avlyst',
+        klikketPaa: false,
+        tilstand: OppgaveTilstand.Ny,
+        sakTittel: '',
+    }),
     oppgave({
         tekst: 'Les og godkjenn avtalen for at den skal kunne tas i bruk',
         klikketPaa: false,

diff --git a/src/mocks/brukerApi/helpers.ts b/src/mocks/brukerApi/helpers.ts
@@ -143,6 +143,7 @@ export const oppgave = ({
     utgaattTidspunkt,
     klikketPaa = true,
     tilleggsinformasjon,
+    lenke = `#${faker.lorem.word()}`,
 }: {
     tekst: string;
     tilstand?: OppgaveTilstand;
@@ -154,6 +155,7 @@ export const oppgave = ({
     utgaattTidspunkt?: Date;
     klikketPaa?: boolean;
     tilleggsinformasjon?: string;
+    lenke?: string;
 }): Oppgave => ({
     __typename: 'Oppgave',
     id: faker.string.uuid(),
@@ -173,7 +175,7 @@ export const oppgave = ({
         klikketPaa,
     },
 
-    lenke: `#${faker.lorem.word()}`,
+    lenke: lenke,
     merkelapp: merkelapp(),
 
     sak:
@@ -193,12 +195,14 @@ export const beskjed = ({
     opprettetTidspunkt = faker.date.recent(),
     klikketPaa = true,
     tilleggsinformasjon,
+    lenke =  `#${faker.lorem.word()}`,
 }: {
     tekst: string;
     sakTittel?: string;
     opprettetTidspunkt?: Date;
     klikketPaa?: boolean;
     tilleggsinformasjon?: string;
+    lenke?: string,
 }): Beskjed => ({
     __typename: 'Beskjed',
     id: faker.string.uuid(),
@@ -213,7 +217,7 @@ export const beskjed = ({
         klikketPaa,
     },
 
-    lenke: `#${faker.lorem.word()}`,
+    lenke: lenke,
     merkelapp: merkelapp(),
 
     sak:

diff --git a/src/utils/amplitude.ts b/src/utils/amplitude.ts
@@ -14,13 +14,10 @@ const createAmpltiudeInstance = (): AmplitudeInstance => {
         .init(getApiKey(), undefined, {
             serverUrl: 'https://amplitude.nav.no/collect',
             useBatch: false,
-            autocapture: {
-                attribution: true,
-                fileDownloads: false,
-                formInteractions: false,
-                pageViews: true,
-                sessions: true,
-                elementInteractions: false,
+            autocapture: false,
+            identityStorage: 'localStorage',
+            trackingOptions: {
+                ipAddress: false,
             },
         })
         .promise.catch((error) => {

diff --git a/src/utils/funksjonerForAmplitudeLogging.ts b/src/utils/funksjonerForAmplitudeLogging.ts
@@ -40,19 +40,19 @@ export const finnBucketForAntall = (
     if (!harRegistrertAntallAnsatte) return '0';
 
     //Hvis harRegistrertAntallAnsatte er true og antall er undefined er det 1-4 ansatte
-    if (antall === undefined) return '1-4';
+    if (antall === undefined) return '1 - 4';
 
     switch (true) {
         case antall < 20:
-            return '5-19';
+            return '5 - 19';
         case antall < 50:
-            return '20-49';
+            return '20 - 49';
         case antall < 100:
-            return '50-99';
+            return '50 - 99';
         case antall < 500:
-            return '100-499';
+            return '100 - 499';
         case antall > 500:
-            return '500>';
+            return '500 >';
         default:
             return undefined;
     }