Merge pull request #206 from navikt/endre-felt-for-tidspunkt-i-kurs

TC-30 fjern fraDato og refaktorer safeSort
navikt · Sep 10, 2021 · 3e48a63 · 3e48a63
2 parents 77ab141 + 8e2dc2f
commit 3e48a63
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 38 deletions.
diff --git a/src/components/paneler/innhold/cv/kurs.tsx b/src/components/paneler/innhold/cv/kurs.tsx
@@ -6,12 +6,11 @@ import { formaterDato, formaterVarighet, safeSort, safeMap } from '../../../../u
 
 function Kurs(props: Pick<ArenaPerson, 'kurs'>) {
 	const { kurs: arenaKurs, ...rest } = props;
-	const sortedKurs = arenaKurs.sort((a, b) => safeSort(b.fraDato, a.fraDato));
+	const sortedKurs = arenaKurs.sort((a, b) => safeSort(b.tidspunkt, a.tidspunkt));
 	const kurs = safeMap(sortedKurs, (enkeltKurs, index) => (
 		<div key={`kurs-${index}`} className="underinformasjon">
 			<Element>{enkeltKurs.tittel}</Element>
 			<Normaltekst>{enkeltKurs.arrangor}</Normaltekst>
-			{enkeltKurs.fraDato && <Normaltekst>Fra: {formaterDato(enkeltKurs.fraDato)}</Normaltekst>}
 			{enkeltKurs.tidspunkt && <Normaltekst>Fullført: {formaterDato(enkeltKurs.tidspunkt)}</Normaltekst>}
 			{enkeltKurs.varighet && <Normaltekst>Varighet: {formaterVarighet(enkeltKurs.varighet)}</Normaltekst>}
 		</div>

diff --git a/src/mock/api/veilarbperson.ts b/src/mock/api/veilarbperson.ts
@@ -1,11 +1,11 @@
-import {rest} from 'msw';
-import {RequestHandlersList} from 'msw/lib/types/setupWorker/glossary';
-import {ArenaPerson, FagdokumentType, KursVarighetEnhet} from '../../rest/datatyper/arenaperson';
-import {PersonaliaInfo} from '../../rest/datatyper/personalia';
-import {AktorId} from '../../rest/datatyper/aktor-id';
-import {Gradering, PersonaliaV2Info} from "../../rest/datatyper/personaliav2";
-import {VergemaalEllerFullmaktOmfangType, VergeOgFullmaktData, Vergetype} from "../../rest/datatyper/vergeOgFullmakt";
-import {TilrettelagtKommunikasjonData} from "../../rest/datatyper/tilrettelagtKommunikasjon";
+import { rest } from 'msw';
+import { RequestHandlersList } from 'msw/lib/types/setupWorker/glossary';
+import { ArenaPerson, FagdokumentType, KursVarighetEnhet } from '../../rest/datatyper/arenaperson';
+import { PersonaliaInfo } from '../../rest/datatyper/personalia';
+import { AktorId } from '../../rest/datatyper/aktor-id';
+import { Gradering, PersonaliaV2Info } from '../../rest/datatyper/personaliav2';
+import { VergemaalEllerFullmaktOmfangType, VergeOgFullmaktData, Vergetype } from '../../rest/datatyper/vergeOgFullmakt';
+import { TilrettelagtKommunikasjonData } from '../../rest/datatyper/tilrettelagtKommunikasjon';
 
 const aktorId: AktorId = {
 	aktorId: '1234567'
@@ -14,7 +14,7 @@ const aktorId: AktorId = {
 const cvOgJobbprofil: ArenaPerson = {
 	sistEndret: '2019-01-15T07:52:35.456+01:00',
 	sammendrag:
-		'Jeg er en maritime executive som har master grad og bachlor grad. Har vart teknisk direktor i mange år og flyttet hjem til Norge hvor jeg søker arbeide innenfor then maritime sektor. Har gode referanser og variert seiling og onshore basert arbeid.',
+		'Jeg er en maritime executive som har mastergrad og bachlorgrad. Har vært teknisk direktor i mange år og flyttet hjem til Norge hvor jeg søker arbeide innenfor then maritime sektor. Har gode referanser og variert seiling og onshore basert arbeid.',
 	arbeidserfaring: [
 		{
 			tittel: 'Maskinsjef',
@@ -101,7 +101,7 @@ const cvOgJobbprofil: ArenaPerson = {
 		{
 			tittel: 'huet',
 			arrangor: 'falk',
-			fraDato: '2016-10',
+			tidspunkt: '2016-10',
 			varighet: {
 				varighet: 1,
 				tidsenhet: KursVarighetEnhet.UKE
@@ -110,18 +110,18 @@ const cvOgJobbprofil: ArenaPerson = {
 		{
 			tittel: 'grønn',
 			arrangor: 'falk',
-			tidspunkt: '2017-10',
+			tidspunkt: '2017-10'
 		},
 		{
 			tittel: 'blå',
 			arrangor: 'falk',
 			fraDato: '2018-10',
-			tidspunkt: '2018-10',
+			tidspunkt: '2018-10'
 		},
 		{
 			tittel: 'dynamik posisjonering',
 			arrangor: 'kongsberg',
-			fraDato: '2010-08'
+			tidspunkt: '2010-08'
 		}
 	],
 	godkjenninger: [
@@ -595,46 +595,46 @@ const mockVergeOgFullmakt: VergeOgFullmaktData = {
 		{
 			motpartsPersonident: '1234567890',
 			motpartsPersonNavn: {
-				fornavn:'Ola',
-				mellomnavn:null,
-				etternavn:'Nordmann',
-				forkortetNavn:'Nordmann Ola'
+				fornavn: 'Ola',
+				mellomnavn: null,
+				etternavn: 'Nordmann',
+				forkortetNavn: 'Nordmann Ola'
 			},
 			motpartsRolle: 'FULLMEKTIG',
 			omraader: [
 				{
-					kode:'AAP',
-					beskrivelse:'Arbeidsavklaringspenger'
+					kode: 'AAP',
+					beskrivelse: 'Arbeidsavklaringspenger'
 				},
 				{
-					kode:'DAG',
-					beskrivelse:'Dagpenger'
+					kode: 'DAG',
+					beskrivelse: 'Dagpenger'
 				}
 			],
 			gyldigFraOgMed: '2021-03-02T13:00:42',
-			gyldigTilOgMed:	'2021-03-03T13:00:42'
+			gyldigTilOgMed: '2021-03-03T13:00:42'
 		},
 		{
 			motpartsPersonident: '1234567891',
 			motpartsPersonNavn: {
-				fornavn:'fornavn',
-				mellomnavn:'mellomnavn',
-				etternavn:'etternavn',
-				forkortetNavn:'forkortetNavn'
+				fornavn: 'fornavn',
+				mellomnavn: 'mellomnavn',
+				etternavn: 'etternavn',
+				forkortetNavn: 'forkortetNavn'
 			},
 			motpartsRolle: 'FULLMAKTSGIVER',
-			omraader:[
+			omraader: [
 				{
-					kode:'BAR',
-					beskrivelse:'Barnetrygd'
+					kode: 'BAR',
+					beskrivelse: 'Barnetrygd'
 				},
 				{
-					kode:'HJE',
-					beskrivelse:'Hjelpemidler'
+					kode: 'HJE',
+					beskrivelse: 'Hjelpemidler'
 				}
 			],
 			gyldigFraOgMed: '2021-03-04T13:00:42',
-			gyldigTilOgMed:	'2021-03-05T13:00:42'
+			gyldigTilOgMed: '2021-03-05T13:00:42'
 		}
 	]
 };

diff --git a/src/rest/datatyper/arenaperson.ts b/src/rest/datatyper/arenaperson.ts
@@ -59,7 +59,6 @@ interface Sprak {
 interface Kurs {
 	tittel: StringOrNothing;
 	arrangor: StringOrNothing;
-	fraDato?: YearMonth;
 	tidspunkt?: YearMonth;
 	varighet?: Kursvarighet;
 }

diff --git a/src/utils/index.ts b/src/utils/index.ts
@@ -91,10 +91,12 @@ export function formaterDato(datoObjekt: DatoType | string | undefined | null, o
 }
 
 export function safeSort(a: StringOrNothing, b: StringOrNothing) {
-	if (a) {
-		return b ? a.localeCompare(b) : -1;
+	if (a && b) {
+		return a.localeCompare(b);
+	} else if (a) {
+		return -1;
 	} else if (b) {
-		return a ? b.localeCompare(a) : 1;
+		return 1;
 	} else {
 		return 0;
 	}