Skip to content

Commit

Permalink
Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
remko48 committed Dec 6, 2024
1 parent c969c11 commit 21b728d
Show file tree
Hide file tree
Showing 8 changed files with 79 additions and 21 deletions.
1 change: 1 addition & 0 deletions src/entities/taak/taak.mock.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ export const mockTaakData = (): TTaak[] => [
toelichting: 'Deze taak omvat het uitvoeren van een gedetailleerde interne audit van de bedrijfsprocessen om te controleren of alle afdelingen voldoen aan de vastgestelde kwaliteitsnormen. De bevindingen worden gedocumenteerd en er worden aanbevelingen gedaan voor verbeteringen.',
actie: 'Voorbereiden van auditchecklist, uitvoeren van audits, rapporteren van bevindingen, aanbevelen van verbeteringen.',
klant: 'urn:uuid:6ec0bd7f-11c0-43da-975e-2a8ad9ebae0b',
medewerker: 'urn:uuid:6ec0bd7f-11c0-43da-975e-2a8ad9ebae0b',
},
]

Expand Down
4 changes: 3 additions & 1 deletion src/entities/taak/taak.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ export class Taak implements TTaak {
public toelichting: string
public actie: string
public klant: string

public medewerker: string
constructor(source: TTaak) {
this.id = source.id || ''
this.title = source.title || ''
Expand All @@ -26,6 +26,7 @@ export class Taak implements TTaak {
this.toelichting = source.toelichting || ''
this.actie = source.actie || ''
this.klant = source.klant || ''
this.medewerker = source.medewerker || ''
}

public validate(): SafeParseReturnType<TTaak, unknown> {
Expand All @@ -40,6 +41,7 @@ export class Taak implements TTaak {
toelichting: z.string(),
actie: z.string(),
klant: z.string(),
medewerker: z.string(),
})

return schema.safeParse(this)
Expand Down
3 changes: 2 additions & 1 deletion src/entities/taak/taak.types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,5 +10,6 @@ export type TTaak = {
onderwerp: string;
toelichting: string;
actie: string;
klant: string;
klant: string
medewerker: string
}
13 changes: 12 additions & 1 deletion src/modals/medewerkers/EditMedewerker.vue
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import { medewerkerStore, navigationStore } from '../../store/store.js'
<NcTextField :disabled="loading"
label="Email adres"
maxlength="255"
:value.sync="medewerkerItem.emailadres" />
:value.sync="medewerkerItem.email" />
</div>

<template #actions>
Expand Down Expand Up @@ -119,6 +119,17 @@ export default {
return this.medewerkerItems
},
},
mounted() {
if (medewerkerStore.medewerkerItem?.id) {
this.medewerkerItem = {
...medewerkerStore.medewerkerItem,
voornaam: medewerkerStore.medewerkerItem.voornaam || '',
tussenvoegsel: medewerkerStore.medewerkerItem.tussenvoegsel || '',
achternaam: medewerkerStore.medewerkerItem.achternaam || '',
email: medewerkerStore.medewerkerItem.email || '',
}
}
},
updated() {
if (navigationStore.modal === 'editMedewerker' && !this.hasUpdated) {
Expand Down
4 changes: 2 additions & 2 deletions src/modals/taken/EditTaak.vue
Original file line number Diff line number Diff line change
Expand Up @@ -55,15 +55,15 @@ import { taakStore, navigationStore, klantStore, medewerkerStore } from '../../s
</NcCheckboxRadioSwitch>

<div>
<NcSelect v-if="clientType !== 'medewerker' || (clientType === 'both' && !useMedewerkerInsteadOfKlant)"
<NcSelect v-if="(clientType !== 'medewerker' && (clientType !== 'both' || clientType === 'both' && !useMedewerkerInsteadOfKlant))"
v-bind="klanten"
v-model="klanten.value"
:user-select="true"
input-label="Klant*"
:loading="klantenLoading"
:disabled="loading" />

<NcSelect v-if="clientType !== 'klant' || (clientType === 'both' && useMedewerkerInsteadOfKlant)"
<NcSelect v-if="(clientType !== 'klant' && (clientType !== 'both' || clientType === 'both' && useMedewerkerInsteadOfKlant))"
v-bind="medewerkers"
v-model="medewerkers.value"
:user-select="true"
Expand Down
10 changes: 6 additions & 4 deletions src/views/medewerkers/MedewerkerDetails.vue
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import { navigationStore, medewerkerStore, taakStore, berichtStore, zaakStore }
<template #icon>
<DotsHorizontal :size="20" />
</template>
<NcActionButton @click="navigationStore.setModal('editKlant')">
<NcActionButton @click="navigationStore.setModal('editMedewerker')">
<template #icon>
<Pencil :size="20" />
</template>
Expand All @@ -40,7 +40,7 @@ import { navigationStore, medewerkerStore, taakStore, berichtStore, zaakStore }
</template>
Zaak starten
</NcActionButton>
<NcActionButton @click="navigationStore.setDialog('deleteKlant')">
<NcActionButton @click="navigationStore.setDialog('deleteMedewerker')">
<template #icon>
<TrashCanOutline :size="20" />
</template>
Expand All @@ -51,8 +51,10 @@ import { navigationStore, medewerkerStore, taakStore, berichtStore, zaakStore }
<span> {{ medewerkerStore.medewerkerItem.subject }} </span>

<div class="gridContent">
<b>Email adres:</b>
<p>{{ medewerkerStore.medewerkerItem.emailadres }}</p>
<div>
<b>Email adres:</b>
<p>{{ medewerkerStore.medewerkerItem.email }}</p>
</div>
</div>
</div>
</div>
Expand Down
9 changes: 2 additions & 7 deletions src/views/medewerkers/MedewerkerList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ import { navigationStore, medewerkerStore } from '../../store/store.js'
:size="44" />
</template>
<template #subname>
{{ getSubname(medewerker) }}
{{ medewerker.email }}
</template>
<template #actions>
<NcActionButton @click="medewerkerStore.setMedewerkerItem(medewerker); navigationStore.setModal('editMedewerker')">
Expand Down Expand Up @@ -119,12 +119,7 @@ export default {
return name
},
getName(medewerker) {
return medewerker?.voornaam ?? 'onbekend'
},
getSubname(medewerker) {
return medewerker?.tussenvoegsel ? `${medewerker.tussenvoegsel} ${medewerker.achternaam}` : medewerker?.achternaam ? `${medewerker.achternaam}` : 'onbekend'
return `${medewerker?.voornaam} ${medewerker?.tussenvoegsel} ${medewerker?.achternaam}`
},
deleteKlant() {
fetch(
Expand Down
56 changes: 51 additions & 5 deletions src/views/taken/TaakDetails.vue
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
<script setup>
import { navigationStore, taakStore, klantStore } from '../../store/store.js'
import { navigationStore, taakStore, klantStore, medewerkerStore } from '../../store/store.js'
</script>

<template>
Expand Down Expand Up @@ -35,7 +35,24 @@ import { navigationStore, taakStore, klantStore } from '../../store/store.js'
<b>Sammenvatting:</b>
<span>{{ taakStore.taakItem.onderwerp }}</span>
</div>
<div>
<div v-if="taakStore.taakItem.medewerker">
<b>Medewerker:</b>
<span v-if="medewerkerLoading">Loading...</span>
<div v-if="!medewerkerLoading" class="buttonLinkContainer">
<span>{{ getMedewerkerName(medewerker) }}</span>
<NcActions>
<NcActionLink :aria-label="`ga naar ${getMedewerkerName(medewerker)}`"
:name="getMedewerkerName(medewerker)"
@click="goToMedewerker()">
<template #icon>
<OpenInApp :size="20" />
</template>
{{ getMedewerkerName(medewerker) }}
</NcActionLink>
</NcActions>
</div>
</div>
<div v-if="taakStore.taakItem.klant">
<b>Klant:</b>
<span v-if="klantLoading">Loading...</span>
<div v-if="!klantLoading" class="buttonLinkContainer">
Expand Down Expand Up @@ -125,22 +142,26 @@ export default {
return {
currentActiveTaak: null,
auditTrails: [],
klant: [],
klant: null,
medewerker: null,
klantLoading: false,
medewerkerLoading: false,
}
},
mounted() {
if (taakStore.taakItem?.id) {
this.currentActiveTaak = taakStore.taakItem
this.fetchAuditTrails(taakStore.taakItem.id)
this.fetchKlant(taakStore.taakItem.klant)
if (taakStore.taakItem.klant) this.fetchKlant(taakStore.taakItem.klant)
if (taakStore.taakItem.medewerker) this.fetchMedewerker(taakStore.taakItem.medewerker)
}
},
updated() {
if (taakStore.taakItem?.id && JSON.stringify(this.currentActiveTaak) !== JSON.stringify(taakStore.taakItem)) {
this.currentActiveTaak = taakStore.taakItem
this.fetchAuditTrails(taakStore.taakItem.id)
this.fetchKlant(taakStore.taakItem.klant)
if (taakStore.taakItem.klant) this.fetchKlant(taakStore.taakItem.klant)
if (taakStore.taakItem.medewerker) this.fetchMedewerker(taakStore.taakItem.medewerker)
}
},
methods: {
Expand All @@ -156,10 +177,17 @@ export default {
getKlantName(klant) {
return klant?.type === 'persoon' ? `${klant?.voornaam} ${klant?.tussenvoegsel} ${klant?.achternaam}` : klant?.bedrijfsnaam
},
getMedewerkerName(medewerker) {
return `${medewerker?.voornaam} ${medewerker?.tussenvoegsel} ${medewerker?.achternaam}`
},
goToKlant() {
klantStore.setKlantItem(this.klant)
navigationStore.setSelected('klanten')
},
goToMedewerker() {
medewerkerStore.setMedewerkerItem(this.medewerker)
navigationStore.setSelected('medewerkers')
},
fetchKlant(klant) {
this.klantLoading = true
Expand All @@ -178,6 +206,24 @@ export default {
})
},
fetchMedewerker(medewerker) {
this.medewerkerLoading = true
fetch(`/index.php/apps/zaakafhandelapp/api/medewerkers/${medewerker}`, {
method: 'GET',
})
.then((response) => {
response.json().then((data) => {
this.medewerker = data
})
this.medewerkerLoading = false
})
.catch((err) => {
console.error(err)
this.medewerkerLoading = false
})
},
},
}
</script>
Expand Down

0 comments on commit 21b728d

Please sign in to comment.