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.

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

131203 - Residence components; update BagData type and email body fun… #2712

Merged
merged 1 commit into from
Nov 25, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
1 change: 1 addition & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -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 (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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" ],
Expand All @@ -76,8 +74,6 @@ const Residence: FC<Props> = ({ caseId }) => {

const woonbootFields = [
[ "Status", woonbootStatus || "–" ],
[ "Indicatie geconstateerd", woonbootIndicatie ],
[ "Aanduiding in onderzoek", woonbootAanduiding ],
<Owner caseData={ caseData } />,
mailtoAnchor
]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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 === ""
Expand Down
8 changes: 2 additions & 6 deletions src/app/features/cases/email/basisInformatie.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import formatBoolean from "app/features/shared/utils/formatBoolean"

Check warning on line 1 in src/app/features/cases/email/basisInformatie.ts

View workflow job for this annotation

GitHub Actions / build

'formatBoolean' is defined but never used

export const email = "Terugmelding.Basisinformatie@amsterdam.nl"

Expand All @@ -14,9 +14,7 @@
etage?: number,
aantalKamers?: number,
oppervlak?: number,
woonbootStatus?: string,
woonbootIndicatie?: boolean,
woonbootAanduiding?: boolean
woonbootStatus?: string
) =>
`Beste collega,

Expand All @@ -36,9 +34,7 @@
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:
Expand Down
19 changes: 9 additions & 10 deletions src/app/features/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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 = {
Expand Down