Merge pull request #2712 from Amsterdam/feature/131203-remove-atlas

131203 - Residence components; update BagData type and email body fun…
Amsterdam · Nov 25, 2024 · e2985ab · e2985ab
2 parents 63f627f + 2429c08
commit e2985ab
Show file tree

Hide file tree

Showing 6 changed files with 38 additions and 48 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/app/features/cases/components/molecules/MailtoAnchor/MailtoAnchor.tsx b/src/app/features/cases/components/molecules/MailtoAnchor/MailtoAnchor.tsx
@@ -29,12 +29,10 @@ const MailtoAnchor: FC<Props> = (
     aantalKamers,
     oppervlak,
     isWoonboot,
-    woonbootStatus,
-    woonbootIndicatie,
-    woonbootAanduiding
+    woonbootStatus
   }
 ) => {
-  const href = `mailto:${ email }?subject=${ subject }&body=${ body(isWoonboot, address, postalCode, gebruiksdoel, gebruik, aantalBouwlagen, etage, aantalKamers, oppervlak, woonbootStatus, woonbootIndicatie, woonbootAanduiding) }`
+  const href = `mailto:${ email }?subject=${ subject }&body=${ body(isWoonboot, address, postalCode, gebruiksdoel, gebruik, aantalBouwlagen, etage, aantalKamers, oppervlak, woonbootStatus) }`
   const text = "Meld BAG afwijkingen"
 
   return (

diff --git a/src/app/features/cases/components/organisms/CaseDetail/sections/Residence.tsx b/src/app/features/cases/components/organisms/CaseDetail/sections/Residence.tsx
@@ -21,45 +21,43 @@ const Residence: FC<Props> = ({ caseId }) => {
 
   const hasBagData = (caseData.bag_data as BagDataError).error === undefined
   const bagData = caseData.bag_data as BagData
-  const isWoonboot = hasBagData && bagData.ligplaatsidentificatie !== undefined
+  const isWoonboot = Boolean(hasBagData && bagData?.ligplaatsIdentificatie)
   const isWoning = !isWoonboot
   const woningTitle = isWoning ? "Woning" : "Ligplaats"
 
   // Woning
-  const gebruiksdoel = hasBagData ? bagData.gebruiksdoel : undefined
+  const gebruiksdoel = hasBagData ? bagData.gebruiksdoelOmschrijvingen : undefined
   const woningBestemming = gebruiksdoel && gebruiksdoel.length ? gebruiksdoel.join(", ") : undefined
-  const woningGebruik = hasBagData && bagData.gebruik ? bagData.gebruik : undefined
-  const woningBouwlagen = hasBagData && bagData.bouwlagen ? bagData.bouwlagen : undefined
-  const woningEtage = hasBagData && bagData.verdieping_toegang != null ? bagData.verdieping_toegang : undefined
-  const woningKamers = hasBagData && bagData.aantal_kamers ? bagData.aantal_kamers : 0
-  const woningOppervlak = hasBagData && bagData.oppervlakte && bagData.oppervlakte > 1 ? bagData.oppervlakte : 0
+  const woningGebruik = hasBagData && bagData.verblijfsobjectFeitelijkGebruikOmschrijving ? bagData.verblijfsobjectFeitelijkGebruikOmschrijving : undefined
+  const woningBouwlagen = hasBagData && bagData.verblijfsobjectAantalBouwlagen ? bagData.verblijfsobjectAantalBouwlagen : undefined
+  const woningEtage = hasBagData && bagData.verblijfsobjectVerdiepingToegang != null ? bagData.verblijfsobjectVerdiepingToegang : undefined
+  const woningKamers = hasBagData && bagData.verblijfsobjectAantalKamers ? bagData.verblijfsobjectAantalKamers : 0
+  const woningOppervlak = hasBagData && bagData.verblijfsobjectOppervlakte && bagData.verblijfsobjectOppervlakte > 1 ? bagData.verblijfsobjectOppervlakte : 0
   const woningBagId = getBagId(caseData!)
 
   // Woonboot
-  const woonbootLigplaatsIndicatie = hasBagData && bagData.ligplaatsidentificatie
-  const woonbootStatus = hasBagData && bagData.status ? bagData.status : undefined
-  const woonbootIndicatie = hasBagData && bagData.indicatie_geconstateerd !== undefined ? bagData.indicatie_geconstateerd : false
-  const woonbootAanduiding = hasBagData && bagData.aanduiding_in_onderzoek !== undefined ? bagData.aanduiding_in_onderzoek : false
+  const woonbootLigplaatsIndicatie = hasBagData && bagData.ligplaatsIdentificatie
+  const woonbootStatus = hasBagData && bagData.ligplaatsStatusOmschrijving ? bagData.ligplaatsStatusOmschrijving : undefined
 
   // General
   const address = getAddress(caseData.address)
   const postalCode = caseData.address.postal_code
 
   // Terugmeld email
-  const mailtoAnchor = <MailtoAnchor
-    address={ address }
-    postalCode={ postalCode }
-    gebruiksdoel={ woningBestemming }
-    gebruik={ woningGebruik }
-    aantalBouwlagen={ woningBouwlagen }
-    etage={ woningEtage }
-    aantalKamers={ woningKamers }
-    oppervlak={ woningOppervlak }
-    isWoonboot={ isWoonboot }
-    woonbootStatus={ woonbootStatus }
-    woonbootIndicatie={ woonbootIndicatie }
-    woonbootAanduiding={ woonbootAanduiding }
-  />
+  const mailtoAnchor = (
+    <MailtoAnchor
+      address={ address }
+      postalCode={ postalCode }
+      gebruiksdoel={ woningBestemming }
+      gebruik={ woningGebruik }
+      aantalBouwlagen={ woningBouwlagen }
+      etage={ woningEtage }
+      aantalKamers={ woningKamers }
+      oppervlak={ woningOppervlak }
+      isWoonboot={ isWoonboot }
+      woonbootStatus={ woonbootStatus }
+    />
+)
 
   const woningFields = [
     // [ "Databron", "BRK" ],
@@ -76,8 +74,6 @@ const Residence: FC<Props> = ({ caseId }) => {
 
   const woonbootFields = [
     [ "Status", woonbootStatus || "–" ],
-    [ "Indicatie geconstateerd", woonbootIndicatie ],
-    [ "Aanduiding in onderzoek", woonbootAanduiding ],
     <Owner caseData={ caseData } />,
     mailtoAnchor
   ]

diff --git a/src/app/features/cases/components/organisms/CaseDetail/utils.ts b/src/app/features/cases/components/organisms/CaseDetail/utils.ts
@@ -7,7 +7,7 @@ export const getBagId = (caseData: Case) => {
   const hasBagData = (caseData?.bag_data as BagDataError).error === undefined
   const bagData = caseData?.bag_data as BagData
 
-  return hasBagData ? (bagData.verblijfsobjectidentificatie ?? bagData.ligplaatsidentificatie) : undefined
+  return hasBagData ? (bagData.verblijfsobjectIdentificatie ?? bagData.ligplaatsIdentificatie) : undefined
 }
 
 export const isNullish = (a: any): a is undefined | null => a === undefined || a === null || a === ""

diff --git a/src/app/features/cases/email/basisInformatie.ts b/src/app/features/cases/email/basisInformatie.ts
@@ -14,9 +14,7 @@ export const body = (
   etage?: number,
   aantalKamers?: number,
   oppervlak?: number,
-  woonbootStatus?: string,
-  woonbootIndicatie?: boolean,
-  woonbootAanduiding?: boolean
+  woonbootStatus?: string
 ) =>
   `Beste collega,
 
@@ -36,9 +34,7 @@ Verdieping toegang: ${ etage !== undefined ? etage : "–" }
 Aantal kamers: ${ aantalKamers !== undefined ? aantalKamers : "–" }
 Woonoppervlak: ${ oppervlak !== undefined ? `${ oppervlak } m² ` : "–" }`
     :
-    `Status: ${ woonbootStatus || "–" }
-Indicatie geconstateerd: ${ formatBoolean(woonbootIndicatie) }
-Aanduiding in onderzoek: ${ formatBoolean(woonbootAanduiding) }`
+    `Status: ${ woonbootStatus || "–" }`
   }
 
 We hebben bij onze controle de volgende afwijkingen geconstateerd:

diff --git a/src/app/features/types.ts b/src/app/features/types.ts
@@ -12,17 +12,16 @@ export type FraudPrediction = {
 }
 
 export type BagData = {
-  aanduiding_in_onderzoek?: boolean
-  aantal_kamers: number | null
-  bouwlagen: number | null
-  gebruik: string | null
-  gebruiksdoel: string[]
+  gebruiksdoelOmschrijvingen: string[]
   indicatie_geconstateerd?: boolean
-  ligplaatsidentificatie?: string
-  oppervlakte: number | null
-  status: string
-  verblijfsobjectidentificatie: string | null
-  verdieping_toegang: number | null
+  ligplaatsIdentificatie?: string
+  ligplaatsStatusOmschrijving?: string
+  verblijfsobjectAantalBouwlagen: number | null
+  verblijfsobjectAantalKamers: number | null
+  verblijfsobjectFeitelijkGebruikOmschrijving: string | null
+  verblijfsobjectIdentificatie: string | null
+  verblijfsobjectOppervlakte: number | null
+  verblijfsobjectVerdiepingToegang: number | null
 }
 
 export type BagDataError = {