Display delete dialog also for collaborations#69
Display delete dialog also for collaborations#69fabiannaether wants to merge 2 commits intodevelopfrom
Conversation
| ) { | ||
| const isParticipant = processModelNode.processType === "PARTICIPANT"; | ||
| if (!skipConfirm && isParticipant) { | ||
| const isProcess = processModelNode.processType === "PROCESS"; |
There was a problem hiding this comment.
Ist das ein Bugfix? Könntest du mir kurz fachlich erklären, worum geht es?
Ich würde auch empfehlen, ProcessModelInformation.processType als Enum zu modellieren (wie im Backend).
There was a problem hiding this comment.
In einer vorherigen Version wurde der Bestätigungsdialog auch beim Löschen von Kollaborationen angezeigt. Damit wurde gezeigt, dass alle Prozessmodelle (= Teilnehmer) der Kollaboration gelöscht werden. Das heißt: In den Fällen processType === "COLLABORATION" oder processType === "PARTICIPANT" gibt es Child- bzw. Parent-Verbindungen. Ich meine, dass die Hinweismeldung am UI auch beim Löschen von Kollaborationen nicht schadet. Nur bei processType === "PROCESS" gibt es keine Verbindungen und damit auch keinen Dialog. Finale Entscheidung, ob das fachlich passt, bitte von @hehnle.
ProcessModelInformation.processType als enum definiert - done
There was a problem hiding this comment.
Danke für die Erklärung!
LGTM
2 participants
Uh oh!
There was an error while loading. Please reload this page.