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

Role time requirements #43

Merged
merged 15 commits into from
Sep 28, 2023
Merged

Conversation

FluffiestFloof
Copy link
Contributor

@FluffiestFloof FluffiestFloof commented Sep 25, 2023

workin on it

About the PR

Currently requiring a decision for time:

  • Keeping the Cadet / Interns as time limited or making them infinitely chosable?
  • Chemist time requirements?
  • Detective time requirements?
  • Paramedic time requirements?
  • Atmos Tech time requirements? Currently only requiring 4 hours as an intern/engineering seems low if antag allowed.
  • Should some civilian role require overall playtime? WD seems to have done so for Bartender, Chef, Mime and Lawyer.

Currently requiring a decision for antag:

  • CMO can no longer be a traitor. Keep or not.
  • Atmos Tech can currently be a traitor. Keep or not.

Roles access left untouched, need to figure out what we want out of them later.

Science and Epistemics needs to be combined

Why / Balance

Technical details

Kept all the roles access untouched. Will require to be decided upon.
Made the executive decision to give CMO 3 hours as Chemist because some of them really need to learn chemistry...
Gave the HoL 2 hours as a Courrier requirement.

Will need to reimplement the whitelist only role system at one point

Media

Breaking changes

Changelog

🆑

  • tweak: Give back all the jobs their proper time requirements.

@github-actions github-actions bot added the Changes: YML Changes any yml files label Sep 25, 2023
@FluffiestFloof
Copy link
Contributor Author

Will need some decision making on certain things marked in the PR's description

@Colin-Tel
Copy link
Contributor

From top to bottom, my opinion:

Cadet/Interns should always be available, maps I think should allow for a larger capacity of these roles
Chemist should require 6 hours (~3 rounds) of medical gametime.
Detective should require 10 hours (~5 rounds) of security gametime. It should also be noted that the plan to have whitelist requirements would likely apply to joining as this role.
Paramedic should require 4 hours (~2 rounds) of medical gametime.
Atmospherics Technician should require 8 hours (~4 rounds) of engineering gametime.

As for civilian roles:
Bartender can have the 30 minute gametime requirement.
Chef can have the 30 minute gametime requirement. ("Noooo I wanna be able to join as a cannibal chef!!")
Mime can have a 2 hour (~1 round) gametime requirement.
Lawyer can have a 10 hour (~5 rounds) general gametime and 4 hour (~2 rounds) security gametime requirement.
Clown can have a 1 minute (~1/120th of a round) gametime requirement. 😁

CMO should be antag-elligible.
HoL should be antag-elligible.
Atmos Tech should be antag-elligible.

@FluffiestFloof
Copy link
Contributor Author

FluffiestFloof commented Sep 27, 2023

Done.
The clown is now time locked 🦊
HoL, CMO, Atmos and Interns can be antags.

@FluffiestFloof FluffiestFloof marked this pull request as ready for review September 27, 2023 17:50
Colin-Tel
Colin-Tel previously approved these changes Sep 28, 2023
@rosieposieeee
Copy link
Contributor

The clown is now time locked 🦊

so real for this

@Colin-Tel
Copy link
Contributor

image

I just want to start a flame in your heart
@FluffiestFloof
Copy link
Contributor Author

-2 for Paramedic, Cargo Technician, Lawyer
-3 for Salvage Specialist, Station Engineer, Technical Assistant
-6 for HoP, HoL, CMO, Mysta
-10 for Atmospheric Technician

@Colin-Tel Colin-Tel merged commit 77a8e1d into DeltaV-Station:master Sep 28, 2023
12 checks passed
DeltaV-Bot pushed a commit that referenced this pull request Sep 28, 2023
@FluffiestFloof FluffiestFloof deleted the timedrfreeman branch October 1, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: YML Changes any yml files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants