Issue-Generator für Vorstandswechsel #24

Merged
MG-95 merged 1 commit from council-change-generator into master 2023-04-26 14:23:47 +02:00
Contributor

Auf https://testredmine.netz39.de könnt ihr die generierten Issues einsehen.
In Organisation-Gruppe sollten es 9 Issues und in der Vorstandsgruppe eine Issue sein.

Wenn ihr weitere Ergänzungen habt, schreibt sie hier ruhig rein.

Auf https://testredmine.netz39.de könnt ihr die generierten Issues einsehen. In Organisation-Gruppe sollten es 9 Issues und in der Vorstandsgruppe eine Issue sein. Wenn ihr weitere Ergänzungen habt, schreibt sie hier ruhig rein.
requested reviews from tux, dkdent, Rick, micha_alb 2023-03-24 17:39:44 +01:00
tux requested changes 2023-03-25 01:42:57 +01:00
tux left a comment
Owner

Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen)

Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen)
councilChange.py Outdated
@ -0,0 +24,4 @@
tracker_id=tracker_ids['admin'],
assigned_to_id=group_ids['admin'])
redmine.issue.create(subject=f"Vorstandswechsel: vorstand@netz39.de anpassen",
Owner

Hier wäre es gut, auch die gewünschte E-Mail-Adresse für den neuen Vorstand zu haben.

Hier wäre es gut, auch die gewünschte E-Mail-Adresse für den neuen Vorstand zu haben.
Author
Contributor

Wollte vermeiden, dass man zum Ausführen des Skript tausend Parameter eingeben muss. Vielleicht wäre es für die Zukunft eine Idee, da direkt Daten aus entities zu extrahieren.

Wollte vermeiden, dass man zum Ausführen des Skript tausend Parameter eingeben muss. Vielleicht wäre es für die Zukunft eine Idee, da direkt Daten aus entities zu extrahieren.
Owner

Das führt jetzt dazu, dass sich der jeweilige Admin diese Infos noch einmal (vom Vorstand) holen muss.

Das führt jetzt dazu, dass sich der jeweilige Admin diese Infos noch einmal (vom Vorstand) holen muss.
Author
Contributor

Da hast du Recht.

Da hast du Recht.
Collaborator

Ich würde das auch so sehen, dass für jede neue Person, die in den Vorstand aufgenommen wird das script einzeln aufgerufen wird und entsprechend für jeden eigene Issues erzeugt.

Ich würde das auch so sehen, dass für jede neue Person, die in den Vorstand aufgenommen wird das script einzeln aufgerufen wird und entsprechend für jeden eigene Issues erzeugt.
Author
Contributor

Gute Idee

Gute Idee
Owner

Dann würde sich das dahingehend ändern, dass es ein Script für das Hinzufügen und eins für das Entfernen von Vorstandsmitgliedern gibt und dann nur das jeweilige Vorstandsmitglied benannt wird.

Dann würde sich das dahingehend ändern, dass es ein Script für das Hinzufügen und eins für das Entfernen von Vorstandsmitgliedern gibt und dann nur das jeweilige Vorstandsmitglied benannt wird.
tux marked this conversation as resolved
councilChange.py Outdated
@ -0,0 +31,4 @@
tracker_id=tracker_ids['admin'],
assigned_to_id=group_ids['admin'])
redmine.issue.create(subject=f"Vorstandswechsel: Signaturen der Mailinglisten anpassen",
Owner

In den Signaturen der Mailinglisten sollten keine Vorstandsnamen stehen. Meinst Du hier allgemein die E-Mail-Signaturen?

In den Signaturen der Mailinglisten sollten keine Vorstandsnamen stehen. Meinst Du hier allgemein die E-Mail-Signaturen?
Author
Contributor

Ich habe nochmal nachgeguckt, wo das auftritt. Es sind die Mail-Signaturen in den Mails, die von @netz39 kommen. Also zum Beispiel tux@netz39.de oder timo@netz39.de

Ich habe nochmal nachgeguckt, wo das auftritt. Es sind die Mail-Signaturen in den Mails, die von @netz39 kommen. Also zum Beispiel tux@netz39.de oder timo@netz39.de
Owner

Okay, die sind aber individuell eingestellt, d.h. das müssen die Leute jeweils in ihren Mail Clients (MUAs) einstellen. Zentral geht das nicht.

Okay, die sind aber individuell eingestellt, d.h. das müssen die Leute jeweils in ihren Mail Clients (MUAs) einstellen. Zentral geht das nicht.
Author
Contributor

Uh das ist aber blöd... Dann sollte man vielleicht abschalten, dass beim Erstellen des Mail-Accounts in Default-Signaturen die Namen des Vorstands vorkommen.

Uh das ist aber blöd... Dann sollte man vielleicht abschalten, dass beim Erstellen des Mail-Accounts in Default-Signaturen die Namen des Vorstands vorkommen.
Owner

Ich habe diese Default-Signaturen noch nicht gesehen. Macht das Roundcube? Der Mail-Server ändert die Mails nicht.

Ich habe diese Default-Signaturen noch nicht gesehen. Macht das Roundcube? Der Mail-Server ändert die Mails nicht.
Author
Contributor

Ja tatsächlich hatte Timo seine Signatur 1:1 von dir kopiert.
Damit ist diese Issue-Erstellung tatsächlich obsolet ;)

Ja tatsächlich hatte Timo seine Signatur 1:1 von dir kopiert. Damit ist diese Issue-Erstellung tatsächlich obsolet ;)
tux marked this conversation as resolved
Author
Contributor

Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen)

Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst.
Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss.

> Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen) Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst. Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss.
Owner

Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen)

Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst.
Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss.

Das überrascht mich - seit wann ist er da nicht mehr drin? Schatzmeister ist ja Teil des Vorstandes.

> > Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen) > > Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst. > Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss. Das überrascht mich - seit wann ist er da nicht mehr drin? Schatzmeister ist ja Teil des Vorstandes.
Author
Contributor

Das überrascht mich - seit wann ist er da nicht mehr drin? Schatzmeister ist ja Teil des Vorstandes.

Grad Micha gefragt. Er bekommt tatsächlich keine Mails von da.

> Das überrascht mich - seit wann ist er da nicht mehr drin? Schatzmeister ist ja Teil des Vorstandes. > Grad Micha gefragt. Er bekommt tatsächlich keine Mails von da.
Collaborator

Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen)

Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst.
Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss.

so weit ich mich erinnere, bekam ich als Schatzmeister diese Mails noch. Zusätzlich dazu die Mails, die an office@netz39.de gehen.

> > Einige der Tickets würden auch den Schatzmeister betreffen (Wechsel Vorstandsliste oder Berechtigungen) > > Nunja, der Schatzmeister ist nicht in vorstand@netz39.de mit drin, falls du das mit Vorstandsliste meinst. > Außerdem würde ich da nen separaten Skript dafür erstellen, weil bei Schatzmeister noch einige andere Dinge getan werden muss. > > so weit ich mich erinnere, bekam ich als Schatzmeister diese Mails noch. Zusätzlich dazu die Mails, die an `office@netz39.de` gehen.
MG-95 force-pushed council-change-generator from d32f04d3b3 to 486a223a19 2023-04-19 14:50:05 +02:00 Compare
Author
Contributor

Habe es grad überbearbeitet:

  • Issue zu Mail-Signaturen entfernt
  • aufgesplittet in Setup und Termination, es werden Skripts für einzelne Personen ausgeführt
  • Mail-Adresse als Parameter eingebaut

Für Schatzmeister-Dinge würde ich einen neuen Branch erstellen.

Die Issues sind wieder in Test-Redmine einsehbar.

Habe es grad überbearbeitet: - Issue zu Mail-Signaturen entfernt - aufgesplittet in Setup und Termination, es werden Skripts für einzelne Personen ausgeführt - Mail-Adresse als Parameter eingebaut Für Schatzmeister-Dinge würde ich einen neuen Branch erstellen. Die Issues sind wieder in Test-Redmine einsehbar.
MG-95 force-pushed council-change-generator from 486a223a19 to 0d0a7ceec0 2023-04-19 14:54:10 +02:00 Compare
requested review from tux 2023-04-19 17:51:13 +02:00
tux approved these changes 2023-04-21 10:31:36 +02:00
tux left a comment
Owner

Ich habe die Scripte nicht getestet, was gemacht wird, sieht im Code okay aus.

Langfristig sollte das ja eh anders getriggert werden, deshalb würde ich das jetzt auch nicht totoptimieren.

Ich habe die Scripte nicht getestet, was gemacht wird, sieht im Code okay aus. Langfristig sollte das ja eh anders getriggert werden, deshalb würde ich das jetzt auch nicht totoptimieren.
MG-95 was assigned by tux 2023-04-21 10:32:05 +02:00
MG-95 merged commit e52613be30 into master 2023-04-26 14:23:46 +02:00
MG-95 deleted branch council-change-generator 2023-04-26 14:23:49 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Netz39_Vorstand/prozesse#24
No description provided.