cozuya / secret-hitler

A web adaptation of the social deduction board game Secret Hitler. https://secrethitler.io
Other
837 stars 202 forks source link

Avalon SH (WIP) #1948

Closed themeeman closed 2 years ago

themeeman commented 2 years ago

Adds two new modes to the game.

Both of these modes have full replay support. Screenshot from 2022-02-22 11-50-39 Screenshot from 2022-02-22 11-52-48 Screenshot from 2022-02-22 11-53-35 Screenshot from 2022-02-22 11-53-52 Screenshot from 2022-02-22 11-53-59 Screenshot from 2022-02-22 11-57-04 Screenshot from 2022-02-22 12-00-45 Screenshot from 2022-02-22 12-00-52 Screenshot from 2022-02-22 12-01-07

iounpaladin commented 2 years ago

Let me know once this is done and I'll review it.

iounpaladin commented 2 years ago

I'm so sorry it took me this long to get to this. It's a large PR and I was pretty busy, but it's finally reviewed. This looks amazing, thank you SO MUCH for putting in the effort to make this, it's incredible. I stress tested everything and apart from my notes everything works perfectly. I have a couple small notes (I initially thought there was a large-scale issue but I realised that it was an issue with my local setup, this also contributed to the time it took to review this):

Should the hitler/fascist text be changed to match as well? Or is it meant to stay the same because of the Hitler election still existing?

These are pretty small issues and I can absolutely take care of them myself if you are busy (you already did so much for this and it is greatly appreciated). Just let me know.

themeeman commented 2 years ago

Thanks for looking :)

  The game begins and you receive the morgana role and take seat 4.
  You see that hitler in this game is ddd {5}. They also see that you are a fascist.

Should the hitler/fascist text be changed to match as well? Or is it meant to stay the same because of the Hitler election still existing?

No I don't think these should be changed, since morgana is still a member of the fascist team, so to say they are "a fascist" would be correct. Hitler is unaffected by this so it should remain the same. Also, hitler wouldn't know which regular fascist is morgana, so saying "they also see you are morgana" would be incorrect.

These are pretty small issues and I can absolutely take care of them myself if you are busy (you already did so much for this and it is greatly appreciated). Just let me know.

Yeah that would be great if you could do them, I can't see it taking long anyway. Thanks.

iounpaladin commented 2 years ago

Perfect, that makes sense. I'll ideally get it done by the end of the week and have it merged by then as well. Thank you again :D