Simplified environment handling in Dolly frontend

#deploy-idporten-frontend #deploy-organisasjon-tilgang-frontend

Removed several dependencies on pdldata for environment determination, simplifying the logic. Refactored how available environments are checked and set for BankID users. The changes impact the Dolly frontend, particularly in the modules DollyFrontendApplicationStarter, MiljoeInfo, Steg3, OrganisasjonTilgangFrontendApplicationStarter, and MiljoVelger.

apps/dolly-frontend/src/main/java/no/nav/dolly/web/DollyFrontendApplicationStarter.java

@@ -131,4 +131,4 @@ private Function<PredicateSpec, Buildable<Route>> createRoute(String segment, St
                         .filters(filter, addUserJwtHeaderFilter())
                 ).uri(host);
     }
-}
+}

apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg3/Steg3.tsx

@@ -28,12 +28,6 @@ export const Steg3 = () => {
 	const importTestnorge = opts.is.importTestnorge
 
 	const erOrganisasjon = formMethods.getValues('organisasjon')
-	const erQ2MiljoeAvhengig =
-		formMethods.watch('pdldata.person.fullmakt') ||
-		formMethods.watch('pdldata.person.falskIdentitet') ||
-		formMethods.watch('pdldata.person.falskIdentitet') ||
-		formMethods.watch('pdldata.person.utenlandskIdentifikasjonsnummer') ||
-		formMethods.watch('pdldata.person.kontaktinformasjonForDoedsbo')
 
 	const bankIdBruker = currentBruker?.brukertype === 'BANKID'
 
@@ -50,9 +44,9 @@ export const Steg3 = () => {
 		if (loading) {
 			return []
 		} else if (bankIdBruker) {
-			return tilgjengeligMiljoe ? [tilgjengeligMiljoe] : ['q1']
+			return tilgjengeligMiljoe ? tilgjengeligMiljoe.split(',') : ['q1']
 		}
-		return erQ2MiljoeAvhengig ? ['q2'] : []
+		return []
 	}
 
 	const erQ1EllerQ2MiljoeAvhengig = (values: any) => {
@@ -73,8 +67,6 @@ export const Steg3 = () => {
 			formMethods.setValue('environments', ['q1', 'q2'])
 		} else if (formMethods.getValues()?.sykemelding) {
 			formMethods.setValue('environments', ['q1'])
-		} else if (erQ2MiljoeAvhengig) {
-			formMethods.setValue('environments', alleredeValgtMiljoe())
 		} else if (!formMethods.getValues()?.environments) {
 			formMethods.setValue('environments', [])
 		}

apps/dolly-frontend/src/main/js/src/components/miljoVelger/MiljoVelger.tsx

@@ -17,19 +17,15 @@ const StyledH3 = styled.h3`
 	align-items: center;
 `
 
-const bankIdQ1 = [
-	{
-		id: 'q1',
-		label: 'Q1',
-	},
-]
+const bankIdQ1 = {
+	id: 'q1',
+	label: 'Q1',
+}
 
-const bankIdQ2 = [
-	{
-		id: 'q2',
-		label: 'Q2',
-	},
-]
+const bankIdQ2 = {
+	id: 'q2',
+	label: 'Q2',
+}
 
 const miljoeavhengig = [
 	'aareg',
@@ -54,13 +50,7 @@ const erMiljouavhengig = (bestilling) => {
 	return miljoeNotRequired
 }
 
-export const MiljoVelger = ({
-	bestillingsdata,
-	heading,
-	bankIdBruker,
-	orgTilgang,
-	alleredeValgtMiljoe,
-}) => {
+export const MiljoVelger = ({ bestillingsdata, heading, bankIdBruker, alleredeValgtMiljoe }) => {
 	const { dollyEnvironments, loading } = useDollyEnvironments()
 	const formMethods = useFormContext()
 
@@ -70,9 +60,20 @@ export const MiljoVelger = ({
 
 	const filterEnvironments = (miljoer, erBankIdBruker) => {
 		if (erBankIdBruker) {
-			const tilgjengeligMiljo = orgTilgang?.miljoe
-			if (tilgjengeligMiljo === 'q1') return bankIdQ1
-			return bankIdQ2
+			var bankMiljoer = []
+			for (var i = 0; i < alleredeValgtMiljoe.length; i++) {
+				switch (alleredeValgtMiljoe[i]) {
+					case 'q1':
+						bankMiljoer.push(bankIdQ1)
+						break
+					case 'q2':
+						bankMiljoer.push(bankIdQ2)
+						break
+				}
+			}
+			console.log('bankMiljoer', bankMiljoer)
+
+			return bankMiljoer
 		}
 		return miljoer.Q.filter((env: any) => env.id !== 'qx')
 	}

apps/dolly-frontend/src/main/js/src/components/miljoVelger/MiljoeInfo.tsx

@@ -1,4 +1,3 @@
-import _ from 'lodash'
 import { Alert } from '@navikt/ds-react'
 import {
 	useArenaEnvironments,
@@ -18,20 +17,8 @@ export const MiljoeInfo = ({ bestillingsdata, dollyEnvironments }) => {
 	} = usePensjonEnvironments()
 	const { instEnvironments, loading: loadingInst, error: errorInst } = useInstEnvironments()
 	const { dokarkivEnvironments, loading: loadingDokarkiv } = useDokarkivEnvironments()
-	const { instdata, pdldata, arenaforvalter, pensjonforvalter, sykemelding, dokarkiv } =
-		bestillingsdata
-	if (
-		!instdata &&
-		!arenaforvalter &&
-		!pensjonforvalter &&
-		!sykemelding &&
-		!dokarkiv &&
-		!_.get(pdldata, 'bostedsadresse') &&
-		!_.get(pdldata, 'fullmakt') &&
-		!_.get(pdldata, 'falskIdentitet') &&
-		!_.get(pdldata, 'utenlandskIdentifikasjonsnummer') &&
-		!_.get(pdldata, 'kontaktinformasjonForDoedsbo')
-	) {
+	const { instdata, arenaforvalter, pensjonforvalter, sykemelding, dokarkiv } = bestillingsdata
+	if (!instdata && !arenaforvalter && !pensjonforvalter && !sykemelding && !dokarkiv) {
 		return null
 	}
 

apps/organisasjon-tilgang-frontend/src/main/java/no/nav/testnav/apps/organisasjontilgangfrontend/OrganisasjonTilgangFrontendApplicationStarter.java

@@ -66,4 +66,4 @@ private Function<PredicateSpec, Buildable<Route>> createRoute(ServerProperties s
                         .filter(filter)
                 ).uri(host);
     }
-}
+}