Project

General

Profile

Project activity #28669

closed

Project activity #28068: Rollen maken voor Tezza | O.b.v. de huidige 4 Alfresco-rollen, omwille dat klanten het beter kunnen beheren

Alleen de rollen Behandelaar en Tezza Beheerder mogen zaakeigenschappen aanpassen

Added by Tahir Malik 11 months ago. Updated 3 months ago.

Status:
Resolved
Priority:
P4 Normal
Category:
-
Target version:
Start date:
27/06/2024
Due date:
% Done:

0%

Estimated time:

Description

Als je nu schrijf rechten hebt op de Zaak/Object/Dossier en je bent geen behandelaar of beheerder kun je zaakeigenschappen aanpassen.

Zie ook rollen & rechten:
https://contezza-my.sharepoint.com/:x:/g/personal/tahir_malik_contezza_nl/EU-0cG3eKjBKvLedgLDMSR0BMghlWRm9vC9CoF3qJcsFRQ?e=kPAK2N

Gewenste situatie is:
  • Alleen de behandelaar of beheerder mag zaakeigenschappen aanpassen

Files

clipboard-202501290856-fb3ru.png (76.6 KB) clipboard-202501290856-fb3ru.png Diego Mirandola, 29/01/2025 08:56 AM
clipboard-202501290905-js7lu.png (42.8 KB) clipboard-202501290905-js7lu.png Diego Mirandola, 29/01/2025 09:05 AM
clipboard-202501290919-qba07.png (17 KB) clipboard-202501290919-qba07.png Diego Mirandola, 29/01/2025 09:19 AM

Related issues 1 (1 open0 closed)

Related to Tezza - Project activity #28358: Autorisatie wijziging voor Behandelaren en TEZZA_BEHEER groep voor afgesloten en gearchiveerde zakenIn ProgressMaaike Bommerson02/05/2025

Actions
Actions #1

Updated by Tahir Malik 10 months ago

  • Parent task changed from #28339 to #28068
Actions #2

Updated by Tahir Malik 7 months ago

  • Description updated (diff)
Actions #3

Updated by Nume Groenewegen 4 months ago

  • Assignee changed from Contezza Development to Nume Groenewegen
Actions #4

Updated by Nume Groenewegen 4 months ago

  • Status changed from Backlog to New
  • Target version changed from Contezza Development Backlog to Tezza 2025.01
Actions #5

Updated by Nume Groenewegen 4 months ago

  • Status changed from New to In Progress
Actions #6

Updated by Maaike Bommerson 3 months ago

  • Related to Project activity #28358: Autorisatie wijziging voor Behandelaren en TEZZA_BEHEER groep voor afgesloten en gearchiveerde zaken added
Actions #7

Updated by Nume Groenewegen 3 months ago

  • Status changed from In Progress to Ready in Dev

Oplossing:

  • De rules voor update & delete zijn uitgebreid met voorwaarden voor de ingelogde gebruiker.
  • De zichtbaarheid van de sidebar voor het toevoegen van zaakeigenschappen is beperkt op basis van de gebruiker en de zaak.
Actions #8

Updated by Nume Groenewegen 3 months ago

  • Assignee changed from Nume Groenewegen to Diego Mirandola

Graag testen voor app tezza in branch feature/tezza/zaakeigenschappen-roles/#28669.

Actions #9

Updated by Diego Mirandola 3 months ago

  • Priority changed from P5 Low to P4 Normal

Updated by Diego Mirandola 3 months ago · Edited

Getest op ZAAK-2025-0000000031 met admin, gorden als behandelaar en paul zaaktypesitemember maar niet behandelaar.
De enige opmerking over het ticket zelf is dat dubbelclick op een eigenschap triggert direct de update action, zonder checks. Twee ideeën hiervoor:
  • ervoor te zorgen dat dit zo werkt:

    Dan wordt of SearchTableLayoutComponent.onMouseEvent of CustomStoreService.doDispatch verantwoordelijk om te herkennen als de action een action type is (huidige gedrag) of een verwijzing naar een extension (let op: ik weet niet als, wanneer deze twee methods worden getriggerd, de nieuwe selectie al gezet is, dus er is een kans dat dit aanpak niet werkt)
  • zoiets implementeren (helper is EffectsHelper)
Opmerkingen over de code:
  • in ZaakeigenschappenTableComponent heb ik dit aangepast (met ViewEncapsulation.None een selector die zo generiek is kan onbewust toegepast worden waar die niet hoort)
  • in ZaakeigenschappenTableComponent, kunnen we niet beter de twee @Input()'s samenvoegen? het wordt dan een @Input() werkmap die beide logica's toepast
  • in ZaakeigenschappenTableComponent, graag naar de rule 'tezza.zaken.zaakeigenschappen.canCreate' verwijzen ipv een kopie van de rule schrijven, anders moeten we de rule in twee plekken beheren
Meer algemene opmerkingen:
  • (ook in de huidige versie) action 'verwijderen' is zichtbaar ook bij multiselectie maar verwijderd alleen een eigenschap, ik heb de rule aangepast
  • mijn test is gebaseerd op dit ticket dat zegt 'Alleen de behandelaar of beheerder mag zaakeigenschappen aanpassen', maar ik zie dat #28358 een verder filtering vraagt op basis van archiefstatus, ik weet niet wat de wens is
  • al validatie is alleen in de UI, al deze actions kunnen onbeperkt direct uitgevoerd worden via contezza-proxy
Actions #11

Updated by Diego Mirandola 3 months ago

  • Assignee changed from Diego Mirandola to Nume Groenewegen
Actions #13

Updated by Nume Groenewegen 3 months ago

  • Status changed from Ready in Dev to In Progress
Actions #14

Updated by Nume Groenewegen 3 months ago

  • Status changed from In Progress to Ready in Dev
  • Assignee changed from Nume Groenewegen to Diego Mirandola

Graag testen in feature/tezza/zaakeigenschappen-roles/#28669

Aanpassingen:
  • filterByRule helper toegevoegd aan effect
  • rule geïmplementeerd in sidebar component
  • rules voor zaakeigenschappen uitgebreid met archived requirements zoals beschreven staat in #28358

LET OP!
Zaakeigenschappen beheren wordt momenteel niet toegestaan door de backend. Nieuwe contezza proxy wijzigingen zijn nodig om het ook daadwerkelijk mogelijk te maken voor admin en tezza beheerders om zaakeigenschappen aan te passen in een gearchiveerde zaak. Met dit tickets is al wel de zichtbaarheid van de actie ingesteld volgens de nieuwe requirements, maar de actie zelf werkt dus nog niet.

Actions #15

Updated by Diego Mirandola 3 months ago · Edited

Getest op ZAAK-2025-0000000033 met admin en gorden. Beschikbaarheid van actions is zoals aangevraagd. Kan mee met de release.

Let op: Opmerking over de code:
  • filterByRule widens het type van de action naar Action, ik heb dit aangepast zodat het specifieke type van de action wordt bewaard.
Actions #16

Updated by Diego Mirandola 3 months ago

  • Status changed from Ready in Dev to Test approved
  • Assignee changed from Diego Mirandola to Nume Groenewegen
Actions #17

Updated by Tahir Malik 3 months ago

  • Target version changed from Tezza 2025.01 to Tezza 2025.02
Actions #18

Updated by Nume Groenewegen 3 months ago

  • Status changed from Test approved to Resolved

In contezza-apps versie 3.20.0

Also available in: Atom PDF