SStalker / EVOS

3 stars 0 forks source link

Prevent of starting the quiz twice #116

Closed lhannigbrinck closed 8 years ago

lhannigbrinck commented 8 years ago

While a quiz is already started it should not be able to start over again at the same time.

SStalker commented 8 years ago

Would it be enough if i change the button to "Quiz beenden" when the quiz is active with route QuizController@beenden to notify the user? Additionally we could deactivate a quiz if it exceed a max active time i.e. 1:30h

(Carbon::now()->timestamp - $quizzes->updated_at->timestamp) > 5400

lhannigbrinck commented 8 years ago

This could be an acceptable solution. I think this issue and #109 are pretty much the same.

cwansart commented 8 years ago

Wie willst du so einen Timer implementieren? Ich würde den Button einfach ausblenden, da man ein Quiz eigentlich nicht beenden kann ohne dass der Nutzer informiert wird, oder?

Ich habe gerade eine Methode implementiert, die den Dozenten informiert, wenn ein Attendee die Session schließt. Da könnte man noch einen Check einbauen, dass wenn die Session dem Dozenten gehört, diese prüft ob er ein aktives Quiz hat und die Teilnehmer informiert oder so.

lhannigbrinck commented 8 years ago

Ausblenden ist eh noch so eine sache. Wurde heute angesprochen un für nicht so toll empfunden, besser wäre da wieder ausgrauen und disablen denke ich.

Ich denke das abbrechen eines quizes könnte man ganz gut in #109 abfangen. Anderersesits sollte das Quiz bei Abbruch durch Disconnect oder dergleichen auch wieder freigegeben werden.

cwansart commented 8 years ago

OK, disable wäre auch in Ordnung. OK, beenden wäre ja nun auch nicht so schlimm, aber das müsste man halte implementieren. Irgendwie überflüssig, oder?

lhannigbrinck commented 8 years ago

Beenden, hmmm aber das ist doch schon einmal implementiert, wobei man dann noch einen end befehl an den sync server schicken müsste.

cwansart commented 8 years ago

Ach so, na gut. :) Dann habe ich nichts gesagt.

lhannigbrinck commented 8 years ago

Oder habe ich das falsch verstanden, @SStalker ?