search

Gepardec / mega

Make the End-of-the-month-process Great Again! - A Gepardec Learning Friday Project
GNU General Public License v3.0
8 stars 2 forks source link

Regex für Projekte bei MEGA aus ZEP #718

Closed WernerBruckmueller closed 7 months ago

WernerBruckmueller commented 10 months ago

  1. What is the current state?

    • [Bug]: Projekte, die eine Projektbeschreibung haben, werden nicht gefunden (weil die Projektbeschreibung? in der Klammer ist).

  2. What is the desired state?

WernerBruckmueller commented 8 months ago

Wenn jemand der ein Kommentar erstellt, einem Projekt nicht zugewiesen ist, so soll es ein Fallback auf Projekt-Intern geben.

LeitnerMi commented 8 months ago

Kontext: Der von mir gefundene Fehler war, dass NUR der Projektleiter einem Mitarbeiter in einem Projekt einen Kommentar erstellen konnte. Keine andere Person konnte einen Kommentar für den Mitarbeiter erstellen.

Technischer Hintegrund: Beim Erstellen des Kommentars, wird nach dem zugehörigen StepEntry gesucht. Dabei wird die Datenanfrage nach assignee-email und owner-email gemacht. Der Owner ist derjenige, der den Kommentar erstellt, der Assignee derjenige der den Kommentar erhält. Da in den Step Entries für die Projekt NUR der Projektleiter als Owner eingetragen ist, wurde dann kein Step Entry gefunden und es konnte kein Kommentar abgelegt werden.

Fix: Datenbankabfrage ändern, sodass nur auf die assignee-email geschaut wird und es egal ist wer den Kommentar erstellen möchte von ZEP aus.

LeitnerMi commented 8 months ago

Aus fachlicher Seite, soll es so sein, dass nicht nur der Projektleiter Kommentare erstellen kann. Aus technischer Sicht? Es gibt keine andere Möglichkeit, als wie den owner des StepEntry zu ignorieren, oder? Es fühlt sich falsch an :sweat_smile: @Ollitod

Ollitod commented 8 months ago

Ich finde es sollten immer nur für diesen MA zuständige Personen Kommentare via ZEP Email erstellen können. Ohne den Check auf den Owner könnte man ja dann das Kommentar zu einem völlig beliebigen StepEntry des MA hinzufügen, was ja keinen Sinn macht oder? In welchem Kontext trat der Fehler auf? @WernerBruckmueller @LeitnerMi FYI

LeitnerMi commented 8 months ago

Beim Testen von 718 hab ich mir das angeschaut und es werden die Projekte eigentlich richtig geparsed. Ich alle durchprobiert mit der Regex und nichts gefunden. Beim Testen bin ich dann auf den "Fehler" gestoßen.

WernerBruckmueller commented 8 months ago

Kein kommentar funktioniert @Ollitod

WernerBruckmueller commented 7 months ago

test erfolgreich