Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Forsøk på å fjerne LoginContext #1711

Merged
merged 9 commits into from
Oct 17, 2023
Merged

Forsøk på å fjerne LoginContext #1711

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
4e64f3a
Fjern LoginContext - kun enkel boundary på topnivå
peterbb Oct 13, 2023
8784e76
Flytt BrowserRouter høyere opp i treet
peterbb Oct 13, 2023
0a0223c
Bruk loader uten widget/bedriftsmeny ved login
peterbb Oct 13, 2023
e43e37c
Merge branch 'master' into fjern_login_context
peterbb Oct 13, 2023
5937d3b
Fiks bad merge og undo urelaterte endringrer
peterbb Oct 16, 2023
905262c
AmplitudeSidevisning er alltid innlogget
ebelegu Oct 16, 2023
935bac8
Fjernet ekstra routes
ebelegu Oct 16, 2023
e6d5169
Merge branch 'fjern_login_context' of https://github.com/navikt/min-s…
ebelegu Oct 16, 2023
4d045a3
Update App.tsx
ebelegu Oct 16, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions src/App/App.css
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,8 @@
justify-content: center;
margin-top: 8rem;
}

.app-finner-ikke-siden {
width: calc(clamp(15rem, 50rem, 100vw - 2rem));
margin: 2rem auto;
}
223 changes: 92 additions & 131 deletions src/App/App.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,14 @@ import React, { FunctionComponent, useContext, useEffect, useState } from 'react
import { BrowserRouter, Route, Link as RouterLink, Routes, useLocation } from 'react-router-dom';
import { basename } from '../paths';
import Hovedside from './Hovedside/Hovedside';
import LoginBoundary from './LoginBoundary';
import { LoginBoundary } from './LoginBoundary';
import { AlertsProvider } from './Alerts/Alerts';
import { OrganisasjonerOgTilgangerProvider } from './OrganisasjonerOgTilgangerProvider';
import { OrganisasjonsDetaljerProvider } from './OrganisasjonDetaljerProvider';
import InformasjonOmBedrift from './InformasjonOmBedrift/InformasjonOmBedrift';
import { ManglerTilgangContainer } from './Hovedside/ManglerTilgangContainer/ManglerTilgangContainer';
import { loggSidevisning } from '../utils/funksjonerForAmplitudeLogging';
import './App.css';
import { Innlogget, LoginContext, LoginProvider } from './LoginProvider';
import { NotifikasjonWidgetProvider } from '@navikt/arbeidsgiver-notifikasjon-widget';
import Banner from './HovedBanner/HovedBanner';
import { Saksoversikt } from './Hovedside/Sak/Saksoversikt/Saksoversikt';
Expand All @@ -29,13 +28,10 @@ const miljø = gittMiljo<'local' | 'labs' | 'dev' | 'prod'>({

const AmplitudeSidevisningEventLogger: FunctionComponent = (props) => {
const location = useLocation();
const { innlogget } = useContext(LoginContext);

useEffect(() => {
if (innlogget !== Innlogget.LASTER) {
loggSidevisning(location.pathname, innlogget);
}
}, [location.pathname, innlogget]);
loggSidevisning(location.pathname);
}, [location.pathname]);

return <>{props.children}</>;
};
Expand All @@ -62,139 +58,104 @@ const App: FunctionComponent = () => {
revalidateOnFocus: false,
}}
>
<LoginProvider>
<LoginBoundary>
<NotifikasjonWidgetProvider
miljo={miljø}
apiUrl={`${basename}/notifikasjon-bruker-api`}
>
<BrowserRouter basename={basename}>
<AmplitudeSidevisningEventLogger>
<Routes>
<Route
path="*"
element={
<LoginBoundary>
<AlertsProvider>
<OrganisasjonerOgTilgangerProvider>
<OrganisasjonsDetaljerProvider>
<Banner sidetittel={sidetittel} />
<Routes>
<Route
path="/bedriftsinformasjon"
element={
<SideTittelWrapper
tittel={
'Om virksomheten'
}
setTittel={
setSidetittel
}
>
<InformasjonOmBedrift />
</SideTittelWrapper>
}
/>
<Route
path="/"
element={
<SideTittelWrapper
tittel={
'Min side – arbeidsgiver'
}
setTittel={
setSidetittel
}
>
<Hovedside />
</SideTittelWrapper>
}
/>
<Route
path="/mangler-tilgang"
element={
<SideTittelWrapper
tittel={
'Min side – arbeidsgiver'
}
setTittel={
setSidetittel
}
>
<ManglerTilgangContainer />
</SideTittelWrapper>
}
/>
<Route
path="/saksoversikt"
element={
<SideTittelWrapper
tittel={'Saksoversikt'}
setTittel={
setSidetittel
}
>
<Brodsmulesti
brodsmuler={[
{
url: '/saksoversikt',
title: 'Saksoversikt',
handleInApp:
true,
},
]}
/>
<Saksoversikt />
</SideTittelWrapper>
}
/>
<Route
path="/sak-restore-session"
element={
<SideTittelWrapper
tittel={'Saksoversikt'}
setTittel={
setSidetittel
}
>
<SaksoversiktRestoreSession />
</SideTittelWrapper>
}
/>
<Route
path="*"
element={
<Alert
style={{
width: 'calc(clamp(15rem, 50rem, 100vw - 2rem))',
margin: '2rem auto',
}}
variant={'error'}
>
{' '}
Finner ikke siden.{' '}
<Link
as={RouterLink}
to={'/'}
>
{' '}
Gå til Min side
arbeidsgiver
</Link>{' '}
</Alert>
}
/>
</Routes>
</OrganisasjonsDetaljerProvider>
</OrganisasjonerOgTilgangerProvider>
</AlertsProvider>
</LoginBoundary>
}
/>
</Routes>
<AlertsProvider>
<OrganisasjonerOgTilgangerProvider>
<OrganisasjonsDetaljerProvider>
<Banner sidetittel={sidetittel} />
<Routes>
<Route
path="/bedriftsinformasjon"
element={
<SideTittelWrapper
tittel={'Om virksomheten'}
setTittel={setSidetittel}
>
<InformasjonOmBedrift />
</SideTittelWrapper>
}
/>
<Route
path="/"
element={
<SideTittelWrapper
tittel={'Min side – arbeidsgiver'}
setTittel={setSidetittel}
>
<Hovedside />
</SideTittelWrapper>
}
/>
<Route
path="/mangler-tilgang"
element={
<SideTittelWrapper
tittel={'Min side – arbeidsgiver'}
setTittel={setSidetittel}
>
<ManglerTilgangContainer />
</SideTittelWrapper>
}
/>
<Route
path="/saksoversikt"
element={
<SideTittelWrapper
tittel={'Saksoversikt'}
setTittel={setSidetittel}
>
<Brodsmulesti
brodsmuler={[
{
url: '/saksoversikt',
title: 'Saksoversikt',
handleInApp: true,
},
]}
/>
<Saksoversikt />
</SideTittelWrapper>
}
/>
<Route
path="/sak-restore-session"
element={
<SideTittelWrapper
tittel={'Saksoversikt'}
setTittel={setSidetittel}
>
<SaksoversiktRestoreSession />
</SideTittelWrapper>
}
/>
<Route
path="*"
element={
<Alert
className={'app-finner-ikke-siden'}
variant={'error'}
>
Finner ikke siden.{' '}
<Link as={RouterLink} to={'/'}>
Gå til Min side arbeidsgiver
</Link>
</Alert>
}
/>
</Routes>
</OrganisasjonsDetaljerProvider>
</OrganisasjonerOgTilgangerProvider>
</AlertsProvider>
</AmplitudeSidevisningEventLogger>
</BrowserRouter>
</NotifikasjonWidgetProvider>
</LoginProvider>
</LoginBoundary>
</SWRConfig>
</div>
);
Expand Down
Loading
Loading