Skip to content

Commit a52cc5f

Browse files
henrikthebosshenriktheboss
authored
Merge branch 'fix-info-tekst-krr' into tester-oppfolging
2 parents 9bbf4ef + 2c1fe65 commit a52cc5f

File tree

9 files changed

+57
-106
lines changed

9 files changed

+57
-106
lines changed

‎.github/workflows/gh-pages.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ jobs:
3838
if: success()
3939
uses: crazy-max/ghaction-github-pages@v2
4040
with:
41+
jekyll: false
4142
build_dir: build
4243
env:
4344
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

‎src/BasePage.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,7 @@
1-
import OppfolgingStatus from './moduler/oppfolging-status/OppfolgingStatus';
21
import { Outlet } from 'react-router-dom';
32

43
export const BasePage = () => {
54
return (
6-
<OppfolgingStatus>
75
<Outlet />
8-
</OppfolgingStatus>
96
);
107
};

‎src/datatypes/eksternAktivitetTypes.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@ type Lenke = RequiredLenkeProps;
1111
type SentimentType = 'POSITIVE' | 'NEUTRAL' | 'NEGATIVE';
1212

1313
export interface Oppgave extends Lenke {
14-
knapptekst: string;
14+
// TODO: Få denne fra backend, den finnes ikke der idag (desverre :( )
15+
knapptekst?: string;
1516
}
1617

1718
export interface LenkeMedType extends RequiredLenkeProps {

‎src/hovedside/Hovedside.tsx

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ import Verktoylinje from '../moduler/verktoylinje/Verktoylinje';
1111
import { useRoutes } from '../routing/useRoutes';
1212
import Aktivitetstavle from './tavle/Aktivitetstavle';
1313
import { Loader } from '@navikt/ds-react';
14+
import OppfolgingStatus from '../moduler/oppfolging-status/OppfolgingStatus';
1415

1516
const Hovedside = () => {
1617
const navigate = useNavigate();
@@ -25,6 +26,8 @@ const Hovedside = () => {
2526
return (
2627
<main id="main" className="hovedside">
2728
<div className="flex items-center flex-col w-full h-full">
29+
<OppfolgingStatus>
30+
</OppfolgingStatus>
2831
<HovedsideFeilmelding />
2932
<InformasjonsHenting />
3033
<Varslinger />

‎src/moduler/aktivitet/visning/eksternaktivitet/OppgaveBoks.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const customAlertStripe = (oppgave: Oppgave) => (
1616
{oppgave.subtekst}
1717

1818
<Button variant={'primary'} as="a" href={oppgave.url} className="flex w-fit mt-4">
19-
{oppgave.knapptekst}
19+
{oppgave.knapptekst || 'Gå videre'}
2020
</Button>
2121
</Alert>
2222
);

‎src/moduler/aktivitet/visning/status-oppdatering/AktivitetStatusForm.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,10 @@ const AktivitetStatusForm = (props: Props) => {
8383

8484
const { setFormIsDirty } = useContext(DirtyContext);
8585

86+
useEffect(() => {
87+
setValue('aktivitet', aktivitet);
88+
}, [aktivitet]);
89+
8690
useEffect(() => {
8791
setFormIsDirty('status', isDirty);
8892
return () => {

‎src/moduler/filtrering/filter/PeriodeFilter.test.tsx

Lines changed: 16 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -118,25 +118,22 @@ const aktiviteterÅpenOgLukketPeriode = (): VeilarbAktivitet[] => {
118118
// ]);
119119
};
120120

121-
const initialStore = (aktiviteter: VeilarbAktivitet[]): RootState =>
122-
mockLoadedStore({
123-
aktiviteter,
124-
arenaAktiviteter: [arenaAktivitet, gammelArenaAktivitet, arenaAktivitetUtenforPeriode],
125-
});
126-
// {
127-
// data: {
128-
// ...emptyHalfLoadedVeilederState.data,
129-
// aktiviteter,
130-
// arenaAktiviteter: {
131-
// status: Status.OK,
132-
// data: [arenaAktivitet, gammelArenaAktivitet, arenaAktivitetUtenforPeriode],
133-
// },
134-
// oppfolging: {
135-
// status: Status.OK,
136-
// data: mockOppfolging,
137-
// },
138-
// },
139-
// }) as unknown as RootState;
121+
122+
const initialStore = (aktiviteter: EntityState<PeriodeEntityState>) =>
123+
({
124+
data: {
125+
...emptyHalfLoadedVeilederState.data,
126+
aktiviteter,
127+
arenaAktiviteter: {
128+
status: Status.OK,
129+
data: [arenaAktivitet, gammelArenaAktivitet, arenaAktivitetUtenforPeriode],
130+
},
131+
oppfolging: {
132+
status: Status.OK,
133+
data: mockOppfolging,
134+
},
135+
},
136+
}) as unknown as RootState;
140137

141138
const lagStore = (initialStore: RootState) =>
142139
configureStore({

‎src/moduler/oppfolging-status/OppfolgingStatus.tsx

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,11 @@
11
import React, { ReactNode } from 'react';
22
import VidereSendBrukereEllerRenderChildren from './VidereSendBrukereEllerRenderChildren';
33

4-
interface Props {
5-
children: ReactNode;
6-
}
74

8-
const OppfolgingStatus = ({ children }: Props) => {
5+
const OppfolgingStatus = () => {
96
return (
107
<div className="w-full">
11-
<VidereSendBrukereEllerRenderChildren>{children}</VidereSendBrukereEllerRenderChildren>
8+
<VidereSendBrukereEllerRenderChildren></VidereSendBrukereEllerRenderChildren>
129
</div>
1310
);
1411
};

‎src/moduler/oppfolging-status/VidereSendBrukereEllerRenderChildren.tsx

Lines changed: 28 additions & 77 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@ import {
1010
selectErBrukerManuell,
1111
selectErRegisrertIKRR,
1212
selectErUnderOppfolging,
13-
// selectKanVarsles,
1413
selectOppfolgingsPerioder,
1514
selectOppfolgingStatus,
1615
selectReservasjonKRR,
@@ -25,34 +24,11 @@ interface VidereSendBrukereEllerRenderChildrenProps {
2524
children: React.ReactNode;
2625
}
2726

28-
function KRRAdvarsel({
29-
// kanVarsles,
30-
erRegistrertIKRR,
31-
erVeilder,
32-
}: {
33-
// kanVarsles: boolean;
34-
erRegistrertIKRR: boolean;
35-
erVeilder: boolean;
36-
}) {
37-
// if (!kanVarsles && erRegistrertIKRR && !erVeilder) {
38-
// return (
39-
// <div className="flex items-center flex-col">
40-
// <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
41-
// <Heading spacing size="small" level="3">
42-
// Kontaktinformasjonen din er utdatert;
43-
// </Heading>
44-
// <p>
45-
// Du kan ikke sende meldinger i dialogen fordi kontaktinformasjonen din er utdatert i kontakt og
46-
// reservasjonsregisteret (KRR).
47-
// </p>
48-
// <Link href={'https://www.norge.no/nb/digital-borgar/oppdatere'}>
49-
// Gå til norge.no for å oppdatere.
50-
// </Link>
51-
// </Alert>
52-
// </div>
53-
// );
54-
// }
55-
if (!erRegistrertIKRR && !erVeilder) {
27+
28+
29+
function KRRAdvarsel({ erRegistrertIKRR, erVeilder}: { erRegistrertIKRR: boolean; erVeilder: boolean; }) {
30+
if (!erRegistrertIKRR && !erVeilder) {
31+
5632
return (
5733
<div className="flex items-center flex-col">
5834
<Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
@@ -69,50 +45,32 @@ function KRRAdvarsel({
6945
</Alert>
7046
</div>
7147
);
72-
} else if (erVeilder && !erRegistrertIKRR) {
73-
return (
74-
<div className="flex items-center flex-col">
75-
<Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
76-
<Heading spacing size="small" level="3">
77-
Brukeren er ikke registrert i KRR
78-
</Heading>
79-
<p>
80-
Du kan ikke bruke aktivitetsplanen fordi brukeren ikke har registrert e-post eller
81-
telefonnummeret sitt i KRR
82-
</p>
83-
<Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>
84-
Brukeren må gå til norge.no for å registrere..
85-
</Link>
86-
</Alert>
87-
</div>
88-
);
48+
49+
} else if (erVeilder && !erRegistrertIKRR){
50+
return(
51+
<div className="flex items-center flex-col">
52+
<Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
53+
<Heading spacing size="small" level="3">
54+
Brukeren er ikke registrert i KRR
55+
</Heading>
56+
<p>
57+
Du kan ikke bruke aktivitetsplanen fordi brukeren ikke
58+
har registrert e-post eller telefonnummeret sitt i KRR
59+
</p>
60+
<Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>
61+
Brukeren må gå til norge.no for å registrere.
62+
</Link>
63+
</Alert>
64+
</div>
65+
)
8966
}
90-
// else if (erVeilder && !kanVarsles){
91-
// return(
92-
// <div className="flex items-center flex-col">
93-
// <Alert variant="warning" className="mx-2 mb-5 max-w-2xl">
94-
// <Heading spacing size="small" level="3">
95-
// Brukeren er ikke registrert i KRR
96-
// </Heading>
97-
// <p>
98-
// Du kan ikke bruke aktivitetsplanen fordi brukeren ikke
99-
// har registrert e-post eller telefonnummeret sitt i KRR
100-
// </p>
101-
// <Link href={'https://www.norge.no/nb/digital-borgar/registrere'}>
102-
// Brukeren må gå til norge.no for å registrere..
103-
// </Link>
104-
// </Alert>
105-
// </div>
106-
// )
107-
// }
10867
}
10968

11069
const VidereSendBrukereEllerRenderChildren = (props: VidereSendBrukereEllerRenderChildrenProps) => {
11170
const underOppfolging = useSelector(selectErUnderOppfolging);
11271
const oppfolgingsPerioder = useSelector(selectOppfolgingsPerioder);
11372
const manuell = useSelector(selectErBrukerManuell);
11473
const reservasjonKRR = useSelector(selectReservasjonKRR);
115-
// const kanVarsles = useSelector(selectKanVarsles);
11674
const erRegistrertIKRR = useSelector(selectErRegisrertIKRR);
11775
const servicegruppe = useSelector(selectServicegruppe);
11876
const aktorId = useSelector(selectAktorId);
@@ -139,23 +97,16 @@ const VidereSendBrukereEllerRenderChildren = (props: VidereSendBrukereEllerRende
13997
) {
14098
return <HarIkkeAktivitetsplan erVeileder={erVeileder} />;
14199
}
142-
if (
143-
//!kanVarsles ||
144-
!erRegistrertIKRR &&
145-
oppfolgingsStatus === Status.OK
146-
) {
147-
return (
148-
<KRRAdvarsel // kanVarsles={kanVarsles}
149-
erRegistrertIKRR={erRegistrertIKRR}
150-
erVeilder={erVeileder}
151-
/>
152-
);
100+
101+
if (!erRegistrertIKRR && oppfolgingsStatus === Status.OK) {
102+
return <KRRAdvarsel erRegistrertIKRR={erRegistrertIKRR} erVeilder={erVeileder}/>;
103+
153104
}
154105
if (ikkeDigitalOppfolging) {
155106
return <AktiverDigitalOppfolging />;
156107
}
157108

158-
return <>{props.children}</>;
109+
return <>{props.children}</>;
159110
};
160111

161112
export default VidereSendBrukereEllerRenderChildren;

0 commit comments

Comments
 (0)