Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(feat) added ashbrook #799

Closed
wants to merge 46 commits into from
Closed

Conversation

starryburst
Copy link
Contributor

new game mode which is like one-night which is a twist on traditional social deduction games like mafia

  • roles currently work (i haven't tested every roles reaction to insanity though)
  • a few things i want to work on before making it playable (demon/minion reveal, voting system, setup creation, a few more roles?)

@starryburst starryburst added enhancement New feature or request frontend/ui React app, front-end descriptions, etc. game/other Other games labels Jul 28, 2023
@starryburst starryburst self-assigned this Jul 28, 2023
@pikulet
Copy link
Contributor

pikulet commented Jul 28, 2023

hmm, how is this different from the one night already on the site

@starryburst
Copy link
Contributor Author

hmm, how is this different from the one night already on the site

it is not one night

- changed secretary + rainmaker text to not confirm secretary on death (follower with self confirming effect)
- added spiker as troublemaker counter (confirmable + solely positive effects)
open roles doesn't work with the game, with demon bluffs, roles that change composition, etc.
- added loudmouth/voucher
- fixed rampager bug
- fixed allCharacters to only have roles in the setup
- fixed poisoned medium bug
@starryburst starryburst requested a review from Fsalker July 31, 2023 21:32
Copy link
Contributor

@Fsalker Fsalker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this tested? is it playable?

leaving this comment here so I know the commit tree at this point

@Fsalker Fsalker marked this pull request as draft August 1, 2023 18:35
@pikulet
Copy link
Contributor

pikulet commented Aug 1, 2023

hey is there a rulebook for how this is played? still confused

@starryburst
Copy link
Contributor Author

so i just added the proper checks for closed role setups (the same as mafia does) so that you will be incapable of creating that setup in the first place so that error would not happen

@starryburst
Copy link
Contributor Author

I'm just putting this here that even though the PR is in draft status it's not currently being worked on as it's waiting to be looked at by someone else (in case people aren't looking at it assuming it's still being worked on and isn't ready)

- changed justice/rainmaker/diabolist role descriptions
- added guiding text (storyteller kill, disperse/come back messages)
- changed exorcism references to trapping
- added non-consecutive targets for attorney
- restorer can't switch kill to a leader
- removed whisper/whisper leak settings in setup creation
- ghoul no longer knows if ability triggered (too powerful for outcast)
- resurrectionist no longer revives outcasts
due to the gamemode being unfamiliar to most players + size of this pr locking a few roles to make it easier for new players to get into + easier reviewing

(13 roles unlocked out of around 50 total)
@starryburst starryburst marked this pull request as ready for review August 15, 2023 02:17
theres pretty much 6 major priority states for night actions, so changed priorities to more accurate reflect that
@DrSharky
Copy link
Contributor

DrSharky commented Aug 23, 2023

I tested it real quick, just a couple games.
I noticed the "you have used your final vote" message shows up multiple times in the system messages.
Typo in the plaguebearer description: should probably say "see" instead of "seem"
Semantic consistency issue in Starseeker description: should probably say "role" instead of "character"
Grammar typo in Starseeker description: Should probably say "player's" instead of "players".

Also, when creating a setup, should the role count for villager, outcast, follower, and leader be visible? I noticed I can see that even if closed role/unique role option is not selected.

Just a few things I noticed so far. I will test more, making sure I understand the game and that it runs well. Then I'll review the code itself.

@starryburst
Copy link
Contributor Author

I tested it real quick, just a couple games. I noticed the "you have used your final vote" message shows up multiple times in the system messages. Typo in the plaguebearer description: should probably say "see" instead of "seem" Semantic consistency issue in Starseeker description: should probably say "role" instead of "character" Grammar typo in Starseeker description: Should probably say "player's" instead of "players".

Also, when creating a setup, should the role count for villager, outcast, follower, and leader be visible? I noticed I can see that even if closed role/unique role option is not selected.

Just a few things I noticed so far. I will test more, making sure I understand the game and that it runs well. Then I'll review the code itself.

for final vote can you show me what happened? i'm not sure how to replicate it

for the role count thing the game is a closed role gamemode (there is no closed role toggle, which is why you see the role counts by default)

starryburst and others added 3 commits August 24, 2023 16:04
removed roles that are not

- destined for initial release
- not destined for initial release but referenced in core game files
- items/effects, etc.
@Golbolco Golbolco closed this Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request frontend/ui React app, front-end descriptions, etc. game/other Other games
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants