Consider all alumni roles when choosing to send mail

hitobito / hitobito_jubla

A hitobito wagon defining the organization hierarchy and additional features for Jungwacht Blauring Schweiz.

Other

15 stars 13 forks source link

Consider all alumni roles when choosing to send mail #46

Closed TheWalkingLeek closed 1 year ago

TheWalkingLeek commented 1 year ago

Fixes: https://github.com/hitobito/hitobito/issues/2121

Wie es scheint erstellen die Wagon Anpassungen beim löschen der letzten Rolle zwei Alumni Rollen:

Eine Rolle in der aktuellen Gruppe, namens Ehemalig. Das ist auch diejenige welche dann auf dem Profil rechts unter "Ehemalige Rollen" angezeigt wird. Im Code alumnus_role genannt.
Eine Rolle in der Ehemaligen-Gruppe des aktuellen Layers. Im Code alumnus_group_member_role genannt.

Weshalb zwei unterschiedliche Rollen erstellt werden verstehe ich inhaltlich nicht. Das ist aber soweit ich das beurteilen kann schon eine Weile so. Die Mails wurden nun nicht versendet da der Mailer Job geprüft hatte ob die Person nur Rollen von dem alumnus_group_member_role "Typ" hat. Da aber immer auch eine vom "Typ" alumnus_role erstellt wurde, war das immer unzutreffend. Mein Fix schaut nun, ob nur noch Rollen vom "Typ" alumnus_group_member_role und/oder alumnus_role vorhanden sind

richardjubla commented 1 year ago

Wie ist der Stand bei diesem Thema? Können wir davon ausgehen, dass das Verhalten auf dem Testsystem funktioniert? Ich habe mehrere Profile ehemalig gemacht und es wurde keine einzige E-Mail an Mailtrap ausgelöst.

Profile: https://jubla.puzzle.ch/groups/296/people/1229.html https://jubla.puzzle.ch/groups/296/people/1246.html

Bei folgendem Profil scheint die "Rolle" einfach verschwunden zu sein: https://jubla.puzzle.ch/groups/166/people/447.html

mlue00 commented 1 year ago

Hallo @richardjubla

Der Pull Request für den Fix ist bereits erstellt. Da wir aber jeweils nach dem Vier-Augen-Prinzip vorgehen und das Review noch nicht stattgefunden hat, wurde dieser noch nicht gemerged. Das heisst, dass das Verhalten auf dem Testsystem noch nicht funktioniert. Aktuell sind bei uns viele Mitarbeitende in den Ferien. Dies verlängert leider die Wartezeit auf das Review. Du wirst natürlich informiert, sobald die Anpassungen auf dem Testsystem verfügbar sind.

Liebe Grüsse Micha

richardjubla commented 1 year ago

Lieber @mlue00

Gibt es einen voraussichtlichen Termin? Ich würde diesen gerne weiter kommunizieren können. Ich habe ehrenamtliche welche sich mit diesem Thema beschäftigen und gerne weiter testen würden.

Liebe Grüsse

ThomasEllenberger commented 1 year ago

Hey @richardjubla. Matthias konnte dies am Freitag noch reviewen und mergen. Das sollte ab sofort auf der Integration wieder funktionieren. Könnt ihr dies testen und uns Rückmeldung geben, ob dies aus eurer Sicht korrekt abgeschlossen ist?

richardjubla commented 9 months ago

Lieber @ThomasEllenberger

Seit dem 9.1 haben wir nun diese Anpassung auf dem Produktiven System. Old News... aber wir haben festgestellt, dass die E-Mails auch auf dem Produktiven System versendet werden. Aus unserer Sicht demzufolge korrekt abgeschlossen.