Merge branch 'master' into bump_apollo_server

navikt · Sep 5, 2023 · c2f7008 · c2f7008
2 parents dad0fd3 + 59ccf1c
commit c2f7008
Show file tree

Hide file tree

Showing 8 changed files with 5,273 additions and 4,790 deletions.
diff --git a/.github/workflows/bygg-og-deploy-master.yml b/.github/workflows/bygg-og-deploy-master.yml
@@ -87,6 +87,7 @@ jobs:
           npx -p @sentry/cli sentry-cli releases new "${{ github.sha }}"
           npx -p @sentry/cli sentry-cli releases files "${{ github.sha }}" upload-sourcemaps build/static/js \
               --url-prefix '~/min-side-arbeidsgiver/static/js'
+          npx -p @sentry-cli sentry-cli releases set-commits --auto "${{ github.sha }}"
           npx -p @sentry/cli sentry-cli releases finalize "${{ github.sha }}"
 
 

diff --git a/nais/dev-gcp.yaml b/nais/dev-gcp.yaml
@@ -15,6 +15,7 @@ spec:
   readiness:
     path: /min-side-arbeidsgiver/internal/isReady
     initialDelay: 20
+  terminationGracePeriodSeconds: 60
   prometheus:
     enabled: true
     path: /min-side-arbeidsgiver/internal/metrics

diff --git a/nais/prod-gcp.yaml b/nais/prod-gcp.yaml
@@ -15,6 +15,7 @@ spec:
   readiness:
     path: /min-side-arbeidsgiver/internal/isReady
     initialDelay: 20
+  terminationGracePeriodSeconds: 60
   prometheus:
     enabled: true
     path: /min-side-arbeidsgiver/internal/metrics

diff --git a/server/package-lock.json b/server/package-lock.json
diff --git a/server/package.json b/server/package.json
@@ -12,7 +12,8 @@
         "express": "^4.17.3",
         "express-http-proxy": "1.6.3",
         "http-proxy-middleware": "3.0.0-beta.1",
-        "jsdom": "^16.4.0",
+        "http-terminator": "3.2.0",
+    "jsdom": "^16.4.0",
         "mustache": "^4.2.0",
         "node-fetch": "^3.2.10",
         "openid-client": "5.1.9",

diff --git a/server/server.js b/server/server.js
@@ -2,6 +2,7 @@ import path from 'path';
 import express from 'express';
 import Mustache from 'mustache';
 import httpProxyMiddleware, { responseInterceptor } from 'http-proxy-middleware';
+import { createHttpTerminator } from 'http-terminator';
 import Prometheus from 'prom-client';
 import { createLogger, format, transports } from 'winston';
 import cookieParser from 'cookie-parser';
@@ -272,10 +273,18 @@ const main = async () => {
         log.info(`Server listening on port ${PORT}`);
     });
 
+    const terminator = createHttpTerminator({
+        server,
+        gracefulTerminationTimeout: 30_000, // defaults: terminator=5s, k8s=30s
+    });
+
     process.on('SIGTERM', () => {
         log.info('SIGTERM signal received: closing HTTP server');
-        server.close(() => {
-            log.info('HTTP server closed');
+        terminator.terminate().then(() => {
+            server.close(() => {
+                log.info('HTTP server closed');
+                process.exit(0);
+            });
         });
     });
 };

diff --git a/src/App/Hovedside/Sak/SisteSaker/SisteSaker.css b/src/App/Hovedside/Sak/SisteSaker/SisteSaker.css
@@ -2,8 +2,7 @@
 .siste_saker {
     background: var(--a-bg-default);
     border-radius: var(--a-border-radius-medium);
-    min-height: 9rem;
-    padding: 1rem 1.5rem;
+    padding: 32px 24px;
     display: flex;
     flex-direction: column;
     gap: 1rem;
@@ -25,7 +24,7 @@
         "ikon undertekst ekspander_kollaps";
     align-items: center;
     min-width: calc(50% - 2.5rem);
-    gap: 0 .5rem;
+    gap: 16px;
 }
 
 @media (max-width: 768px) {
@@ -80,13 +79,4 @@
 
 .saker-lenke__undertekst {
     grid-area: undertekst;
-    white-space: nowrap;
-    overflow: hidden;
-    text-overflow: ellipsis;
-}
-
-.saker_lenke__undertekst_ekspandert {
-    white-space: normal;
-    overflow: visible;
-    text-overflow: unset;
 }
diff --git a/src/App/Hovedside/Sak/SisteSaker/SisteSaker.tsx b/src/App/Hovedside/Sak/SisteSaker/SisteSaker.tsx
@@ -5,11 +5,10 @@ import './SisteSaker.css';
 import { useSaker } from '../useSaker';
 import { loggNavigasjon } from '../../../../utils/funksjonerForAmplitudeLogging';
 import amplitude from '../../../../utils/amplitude';
-import { BodyShort, Button, Heading } from '@navikt/ds-react';
+import { BodyShort, Heading } from '@navikt/ds-react';
 import { useSessionStateForside } from '../Saksoversikt/useOversiktSessionStorage';
-import { OppgaveTilstand, SakSortering } from '../../../../api/graphql-types';
-import { Collapse, Expand } from '@navikt/ds-icons';
-import { BellDotFillIcon, PaperplaneIcon } from '@navikt/aksel-icons';
+import { SakSortering } from '../../../../api/graphql-types';
+import { PaperplaneIcon } from '@navikt/aksel-icons';
 import { OrganisasjonerOgTilgangerContext } from '../../../OrganisasjonerOgTilgangerProvider';
 import { sorted } from '../../../../utils/util';
 import { Set } from 'immutable';
@@ -21,74 +20,62 @@ interface SakerLenkeProps {
     overskrift: string;
     undertekst: string;
     to: To;
-    ekspander: boolean;
-    setEkspander: (ekspander: boolean) => void;
 }
 
-const SakerLenke = ({ ikon, overskrift, undertekst, to, ekspander, setEkspander}: SakerLenkeProps) => {
+const SakerLenke = ({ ikon, overskrift, undertekst, to }: SakerLenkeProps) => {
     const [hover, setHover] = useState(false);
-    const ikonId = "ikon-id" + overskrift.toLowerCase().replace(' ', '-');
-    const lenkeId = "lenke-id" + overskrift.toLowerCase().replace(' ', '-');
+    const ikonId = 'ikon-id' + overskrift.toLowerCase().replace(' ', '-');
+    const lenkeId = 'lenke-id' + overskrift.toLowerCase().replace(' ', '-');
 
     addEventListener('mouseover', (e) => {
         const target = e.target as HTMLElement;
         if (target.id === ikonId) {
             setHover(true);
-        }
-        else if (target.id === lenkeId) {
+        } else if (target.id === lenkeId) {
             setHover(true);
-        }
-        else {
+        } else {
             setHover(false);
         }
-    })
-    return <div className='saker-lenke'>
-        <Link
-            tabIndex={-1}
-            id={lenkeId}
-            className={'saker-lenke__ikon' + (hover ? ' saker-lenke__ikon__hover' : '')}
-            to={to}
-            onClick={() => {
-                scroll(0, 0);
-                loggNavigasjon('saksoversikt', 'se alle saker', location.pathname);
-            }}
-        >
-            {ikon}
-        </Link>
-
-        <Link className={'saker-lenke_headerlenke' + (hover ? ' saker-lenke_headerlenke__hover' : '')}
-              to={to}
-              onClick={() => {
-                  scroll(0, 0);
-                  loggNavigasjon('saksoversikt', 'se alle saker', location.pathname);
-              }}
-        >
-            <Heading id={ikonId} size={'small'}>
-                {overskrift}
-            </Heading>
-        </Link>
-        <BodyShort
-            className={'saker-lenke__undertekst ' + (ekspander ? ' saker_lenke__undertekst_ekspandert' : " ")}
-        >{undertekst}</BodyShort>
-        <Button
-            aria-label={`${ekspander ? "Skjul deler av" : "Vis hele"} teksten. (Hele teksten vises for skjermleser)`}
-            className="saker-lenke__ekspander-knapp"
-            size="xsmall"
-            variant="tertiary"
-            style={{color: "black"}}
-            onClick={() => setEkspander(!ekspander)
-        }
-        >
-            {ekspander ? <Collapse aria-hidden/> : <Expand aria-hidden/>}
-        </Button>
-
-    </div>;
+    });
+    return (
+        <div className="saker-lenke">
+            <Link
+                tabIndex={-1}
+                id={lenkeId}
+                className={'saker-lenke__ikon' + (hover ? ' saker-lenke__ikon__hover' : '')}
+                to={to}
+                onClick={() => {
+                    scroll(0, 0);
+                    loggNavigasjon('saksoversikt', 'se alle saker', location.pathname);
+                }}
+            >
+                {ikon}
+            </Link>
+
+            <Link
+                className={
+                    'saker-lenke_headerlenke' + (hover ? ' saker-lenke_headerlenke__hover' : '')
+                }
+                to={to}
+                onClick={() => {
+                    scroll(0, 0);
+                    loggNavigasjon('saksoversikt', 'se alle saker', location.pathname);
+                }}
+            >
+                <Heading id={ikonId} size={'small'}>
+                    {overskrift}
+                </Heading>
+            </Link>
+            <BodyShort className="saker-lenke__undertekst ">{undertekst}</BodyShort>
+        </div>
+    );
 };
 
 const SisteSaker = () => {
-    const { valgtOrganisasjon, antallSakerForAlleBedrifter } = useContext(OrganisasjonsDetaljerContext);
+    const { valgtOrganisasjon, antallSakerForAlleBedrifter } = useContext(
+        OrganisasjonsDetaljerContext
+    );
     const { organisasjoner } = useContext(OrganisasjonerOgTilgangerContext);
-    const [ekspander, setEkspander] = useState(false);
     const location = useLocation();
 
     const { loading, data } = useSaker(ANTALL_FORSIDESAKER, {
@@ -100,17 +87,6 @@ const SisteSaker = () => {
         oppgaveTilstand: [],
     });
 
-    const sakerMedOppgaverRespons = useSaker(ANTALL_FORSIDESAKER, {
-        side: 1,
-        virksomheter: Set(),
-        tekstsoek: '',
-        sortering: SakSortering.Frist,
-        sakstyper: [],
-        oppgaveTilstand: [OppgaveTilstand.Ny],
-    });
-
-    const sakerMedOppgaver = sakerMedOppgaverRespons.data?.saker
-
     useSessionStateForside();
 
     useEffect(() => {
@@ -128,39 +104,29 @@ const SisteSaker = () => {
 
     if ((antallSakerForAlleBedrifter ?? 0) === 0) return null;
 
-    const antallVirksomheter = Object.values(organisasjoner).filter(org => ["BEDR", "AAFY"].includes(org.organisasjon.OrganizationForm)).length;
+    const antallVirksomheter = Object.values(organisasjoner).filter((org) =>
+        ['BEDR', 'AAFY'].includes(org.organisasjon.OrganizationForm)
+    ).length;
 
     return (
-        <div className='siste_saker'>
-            <Heading size='small' level='2'> Saker {antallVirksomheter > 1 ? "for dine virksomheter" : "" } </Heading>
-            <div className='siste_saker_valg_container'>
+        <div className="siste_saker">
+            <div className="siste_saker_valg_container">
                 <SakerLenke
-                    to = {{
+                    to={{
                         pathname: 'saksoversikt',
                         search: location.search,
                     }}
-                    ikon={<PaperplaneIcon aria-hidden/>}
-                    overskrift={`Antall saker (${antallSakerForAlleBedrifter})`}
-                    undertekst={sorted(data.saker.sakstyper, sakstype => sakstype.navn).map((sakstype) => `${sakstype.navn} ${sakstype.antall}`).join(', ')}
-                    ekspander={ekspander}
-                    setEkspander={setEkspander}
-                />
-                <SakerLenke
-                    to = {{
-                        pathname: 'saksoversikt',
-                        search: location.search + "&oppgaveTilstand=NY"
-                    }}
-                    ikon={<BellDotFillIcon aria-hidden/>}
-                    overskrift={'Med oppgaver ' + (((sakerMedOppgaver?.totaltAntallSaker ?? 0) > 0) ? `(${sakerMedOppgaver?.totaltAntallSaker})` : '')}
-                    undertekst={sorted(sakerMedOppgaver?.sakstyper ?? [], sakstype => sakstype.navn).map(
-                        (sakstype) => `${sakstype.navn} ${sakstype.antall}`).join(', ')
-                    }
-                    ekspander={ekspander}
-                    setEkspander={setEkspander}
+                    ikon={<PaperplaneIcon aria-hidden />}
+                    overskrift={`Alle saker ${
+                        antallVirksomheter > 1 ? 'for dine virksomheter' : ''
+                    } (${antallSakerForAlleBedrifter})`}
+                    undertekst={sorted(data.saker.sakstyper, (sakstype) => sakstype.navn)
+                        .map((sakstype) => `${sakstype.navn} ${sakstype.antall}`)
+                        .join(', ')}
                 />
             </div>
         </div>
     );
 };
 
-export default SisteSaker;
+export default SisteSaker;