search

fs-geofs / ErstiWeTool

signup & management platform for our freshmen's weekend
GNU General Public License v3.0
0 stars 1 forks source link

fix Issue 10 #15

Closed noerw closed 6 years ago

noerw commented 6 years ago

last year, haben sich ein paar spaete voegel auf die warteliste gesetzt, aber alle frei gewordenen token gingen bereits vorher an die warteliste raus. solche menschen werden nun direkt informiert ueber freie token.

problem: solange nicht alle token am anfang eingeloest sind, kann man sich einen erschleichen indem man sich direkt auf die warteliste setzt.. sollen wir die hier hinzugefuegt funktion ueber eine config flag explizit erst anmachen?

SpeckiJ commented 6 years ago

solche menschen werden nun direkt informiert ueber freie token.

Wird das random 8-24h Timeout auch fuer jemanden der sich erst spaeter auf die Liste setzt enforced? Ansonsten kann man die ganze Warteliste ja skippen wenn man sich gut abspricht. (Person A meldet sich ab, Person B schreibt sich 2min spaeter auf die WL)

config flag

Zeitgesteuert dann aus der config.js? Oder meinst du wirklich command-line parameter?

christophfriedrich commented 6 years ago

Wie wär's, wenn das Tool erst dann Einträge auf die Warteliste erlaubt, wenn alle Tokens eingelöst sind? Das kann man ja easy in der DB abfragen.

noerw commented 6 years ago

@SpeckiJ es werden keine token direkt rausgegeben solange noch welche scheduled sind. es geht ja um denn fall dass keine mehr scheduled sind, aber trotzdem noch plaetze frei sind.

@christophfriedrich gute idee eigentlich. koennen wir sicher sein, dass bei der fruehstuecksanmeldeschlange alle token direkt eingeloest werden?

noerw commented 6 years ago

Ok, andere Loesung: ueber die Warteliste sollten nur Nachruecktoken rausgehen, dh wir koennen pruefen ob Nachruecktoken frei sind, nicht Originaltoken

christophfriedrich commented 6 years ago

koennen wir sicher sein, dass bei der fruehstuecksanmeldeschlange alle token direkt eingeloest werden?

Entsprechend der Erfahrungen der letzten Jahre JA, aber 100% kann man das natürlich nicht garantieren.

Deine Idee, Originaltoken einfach nicht zu berücksichtigen, finde ich daher eine gute Lösung.

noerw commented 6 years ago

ok, dann lassen wirs so wie hier implementiert. @SpeckiJ ich deploye das dann gleich, aber lass uns das ding nochmal durchtesten in den naechsten tagen

SpeckiJ commented 6 years ago

@noerw

jo gerne, aber bitte die Tokens nicht neu generieren - die sind alle schon fertig ausgedruckt und zugeschnitten :)