Contezza Community » Tezza

Koppelen aan correct project

Zie analyse hieronder, graag doorzetten naar degene die dit verder moet oppakken of bespreken bij de volgende Tezza overleg.

Analyse / refactoring in de app (branch feature/tezza/roltoelichting/#32324)

• action 'rolCreate'

'roltoelichting' wordt al niet opgestuurd in de payload voor betrokkenen van type medewerker, dit wordt alleen ingevuld door de backend (vereist door zaken api specificatie)
'roltoelichting' verwijderd vanuit de payload van alle haal-centraal koppelingen (brp, hr, iconnect, opentunnel) voor alle betrokkene typen (np, nnp, vestiging)
veld 'roltoelichting' gemarkeerd als optioneel in het model van de payload

Let op: action wordt correct uitgevoerd, veld 'roltoelichting' wordt ingevuld door de backend (vereist door zaken api specificatie), met een andere structuur dan degene van de app. Dit wordt een probleem bij 'rolRemove'. Bovendien is het onhandig dat voor medewerkers wordt de dbid en niet de username opgeslagen.

• action 'rolRemove'

de backend checkt: rol.getRoltoelichting().contains(userName); om te begrijpen welke rol verwijderd moet worden; hierdoor kan de action niet meer werken voor np en nnp
Waarom werkt de backend niet op basis van de uuid van de role?

Anders moet RollenService.getAuthority() refactored worden om niet op basis van 'roltoelichting' te werken, en dezelfde data vanuit 'betrokkeneIdentificatie' te lezen. Dit is makkelijk voor np, nnp, vestiging maar voor medewerkers gebruikt dit nu de username, in betrokkeneIdentificatie wordt de dbid opgeslagen, dit is onhandig

• aanwezigheid van property 'roltoelichting' wordt door de code gebruikt om te herkennen dat een object een rol is. Dit blijft correct want 'roltoelichting' is verplicht in zaken api

• andere gebruik van 'roltoelichting' in de app is vervangen door RollenService.getAuthority()

Daarnaast:

• pipe ozInvolvedName verwijderd want niet gebruikt