hitobito / hitobito_jubla

A hitobito wagon defining the organization hierarchy and additional features for Jungwacht Blauring Schweiz.
Other
15 stars 13 forks source link

GROUPS: Fehler beim Fusionieren von Gruppen #31

Closed mtnstar closed 2 years ago

mtnstar commented 2 years ago

Aktuelles Verhalten

Teilweise können zwei Gruppen nicht fusioniert werden. HTTP Error Code 422, 500

Gewünschtes Verhalten

Beliebige Gruppen können fusioniert werden.

Schritte zum reproduzieren

  1. Auf dem Info-Tab einer Gruppe unter Bearbeiten die Option Fusionieren wählen
  2. Name für die neue Gruppe sowie die Gruppe mit welcher fusioniert werden soll auswählen
  3. "Gruppe fusionieren" wählen

Tech-Spec

mtnstar commented 2 years ago

es gibt eine Rolle an der keine Peson assigned ist: Role.id 355481. Beim Löschen dieser Rolle kam es dabei zu einem Fehler (beim Versuch die primary_group_id auf der Person zu aktualisieren)

ich habe die entsprechende Rolle auf der Produktion gelöscht und zuvor die Fusion bei mir lokal getestet. Scheint zu funktionieren.

@Richard willst du nochmals versuchen die beiden Gruppen zu fusionieren?

richardjubla commented 2 years ago

Wir haben wieder/weiterhin Fehlermeldungen bei der Fusion von Gruppen. https://help.puzzle.ch/#ticket/zoom/3517

Mir fehlen die groben Kriterien nach denen wir suchen sollen, um Validierungsfehler für eine Fusion zu erkennen.

richardjubla commented 2 years ago

Error 422 bedeutet Validation Fehler. Für den User: Datenbank kann deine Fusion nicht durchführen, da gewisse Daten fehlen oder nicht korrekt vorhanden sind.

Wir validieren Gruppen auf folgende Aspekte: Name: Muss vorhanden sein E-Mail: Format einer Mailadresse, Domain muss erreichbar sein (DNS-Eintrag) Beschreibung: Maximallänge: 2^16 - 1 (= 65535) Zeichen Adresse: Maximallänge: 1024 Zeichen Kontaktperson: Muss in der Gruppe sein Land: Muss im Dropdown vorhanden sein PLZ: Muss eine Schweizer PLZ sein

Anhand dieser Aspekte können wir prüfen oder klären, ob der Fehler eher in den Daten oder in der Technik zu vermuten ist.