welcomehelpde / engelsystem

Fork und Anpassung engelsystem von welcomehelpde 7.10.15
GNU General Public License v2.0
10 stars 6 forks source link

Benutzer automatisch auf "aktiv" setzen nach erfolgreichem Double Opt-in #57

Open netrunner99 opened 8 years ago

netrunner99 commented 8 years ago

guest_login.php:user_activate_account_controller

vahldiek commented 8 years ago

Bist du hier bereits bei der Implementierung? (Da WIP bereits gesetzt?) Ansonsten koennte ich versuchen das schnell mal einzufuegen...

sdvg commented 8 years ago

Soweit ich weiß ist da noch niemand dran.

vahldiek commented 8 years ago

ok, sollte schnell gehen... in den naechsten tagen...

vahldiek commented 8 years ago

Es scheint noch einige Reliquiuen aus dem alten CCC system in diesem Bereich. Kommt sicherlich daher, dass die meisten Helfer erst zu einem Zeitpunkt ankommen und unter Umstaenden vor Ende der Konferenz Abreisen. Wichtig in den bisher impelentierten Abfragen scheint das User bit 'Gekommen' zu sein, deshalb wird das 'Gekommen' bit und die dazugehoerige Zeit eingetragen.

Hattet ihr gewisse Features im Blick, die aktiv werden, wenn der User auf Aktiv gesetzt ist?

netrunner99 commented 8 years ago

Dieser Mechanismus, erst vom Admin aktiv gesetzt zu werden, bevor der Helfer etwas aktives tun kann , wie sich in eine Schicht einzutragen .. im Engelsystem .. hat durchaus seine Berechtigung. So könnte man ggf. Störer erkennen und deren Aktionen im voraus verhindern oder bremsen, Doch setzt dies eine Adminaktion zwingend voraus bei jeder Neuanmeldung und kann zu viel Arbeit für die Admins ausarten und verhindert eine schnelle Anmeldung und Eintragung neuer Freiwilligen.

Es ist die Frage, ob hier ein guter Kompromiss gefunden werden kann.

vahldiek commented 8 years ago

Soweit ich dich verstehe, braucht man dann eine manuellen Barriere. Der merge request sollte in diesem Fall nicht angenommen werden, denn dann sind alle Tore offen. Jeder kann sich zu Schichten anmelden, die keine eingescrhaenkte Groppe voraussetzt.

Vorschlag A: Wisst ihr ob die Admins in der Praxis eintragen wer erschienen ist und wer nicht? Wenn dies eingetragen wuerde hatte ich einen weiteren Vorschlag. 0) Neuanmeldungen sind nicht aktiv 1) Jeder kann sich zu Schichten anmelden fuer die er Gruppenrechte besitzt 2) Die Anzahl der benoetigten Helfer pro Schicht wird nur fuer aktive Nutzer reduziert 3) Wenn der Helfer zur besagten Schicht erscheint, wird er auf aktiv gesetzt (falls er nicht schon vorher an einer Schicht teilgenommen hat)

Dadruch haetten offentliche Personen nicht die Moeglichkeit die gebrauchten Helferzahlen zu manipulieren. (also nur, wenn sie Zugriff zu Accounts haetten, die bereits mind. einmal Vorort waren)

Vorschlag B: Anstatt mit einem 2 Klassensystem zu arbeiten, koennte man die Stoerer begrenzen. Zum Beispiel waere denkbar, dass nur pro inaktiven Account im Voraus eine Schicht angemeldet werden kann.

Waere an eueren Vorschlaegen und Feedback interessiert. Sehr wahrscheinlich sollte das Verhalten des Anmeldeprozesses auch configurierbar sein und nicht statisch im Code geaendert werden muessen...

netrunner99 commented 8 years ago

Vorschlag A) sehr ähnliches wurde schon im RealLife mit Papierlisten umgesetzt mit Erfolg. Allerdings wollen nicht alle Standorte diesen Aufwand treiben mit dem manuellem Aktiv-Setzen vor jedem Einsatz vorort durch den Schichtleiter.

Vorschlag B) klingt nach einem vielversprechenden Kompromiss!

Auch an die Konfigurierbarkeit habe ich schon gedacht.

Vielen Dank für die guten Ideen!