]> git.cworth.org Git - empires-server/commitdiff
Extend PHASE_IDLE_TIMEOUT from 10 seconds to 30 seconds
authorCarl Worth <cworth@cworth.org>
Thu, 18 Jun 2020 14:59:17 +0000 (07:59 -0700)
committerCarl Worth <cworth@cworth.org>
Fri, 26 Jun 2020 14:37:59 +0000 (07:37 -0700)
The 10 second value was always known to be aggressively short. I had
this in there with such a short value because we didn't have the
feature to auto-advance a phase after every single player was in, nor
to auto-idle the phase after the players-waiting list is emptied. So
without those features implemented, players were forced to wait for
the idle timeout in situations where there was nothing to wait for. So
to keep that from being too annoying, I had this parameter dialed
down.

But now that those two missing features were implemented in recent
commits, we take this timeout up to a higher value. With this,
hopefully the Move On button won't appear too soon so that people
inadvertently skip an active player, but instead only appears once
people really are stuck waiting.

empathy.js

index 354e3fac2e71838a8753ac71526af40363095782..28b30638735b25839d5c4023dec7c26a4b6e0d0f 100644 (file)
@@ -19,7 +19,7 @@ const PHASE_MINIMUM_TIME = 30;
  *
  * Specified in seconds
  */
-const PHASE_IDLE_TIMEOUT = 10;
+const PHASE_IDLE_TIMEOUT = 30;
 
 class Empathy extends Game {
   constructor(id) {