-
-
Notifications
You must be signed in to change notification settings - Fork 110
Doomsayer Rework #1470
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
base: dev_2.3.0
Are you sure you want to change the base?
Doomsayer Rework #1470
Conversation
In fact, I think that this is a very inefficient code that has 2 significant disadvantages When adding a new role, probably no one will add the new role to the Doomsayer list Even if someone adds a role to the list, they will also have to change the lines and this will reset the current lines in all languages, which is very annoying to change this every time |
Previously, FortuneTeller had exactly the same code where each role was added manually and was eventually changed so that it selected random roles automatically |
So, I believe that the code should show the roles the same as in TOU-R when it selects a player and from the randomly active roles only one is correct Or does it work somehow differently and I don’t remember how Doomsayer in TOU-R works |
TOU-R does the same thing I've done, roles are put into categories for doomsayer and are hard-coded, which is fine because that mod doesn't have as much roles |
I can perfectly understand why fortune Teller was reworked to the way it is now but if I did the same thing with doomsayer that would make the role way too easy |
It might be better to make the code check each role via |
the issue with that is that im not sorting it by |
with a few exceptions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request reworks the Doomsayer role by adding an observation feature in Easy Mode and updating related game options and messaging. Key changes include:
- Adding new options (Easy Mode and Observe Cooldown) and associated UI elements.
- Implementing behavior for observation via the kill button and updating message handling.
- Updating localization strings to support the new observation hints.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
File | Description |
---|---|
Roles/Neutral/Doomsayer.cs | Added Easy Mode functionality with observation logic, new option items, and updated behavior. |
Resources/Lang/en_US.json | Updated language strings to include descriptions for Easy Mode and observation messages. |
(Defaults to off)
Doomsayer can now observe a player to get a hint about their role upon the next meeting