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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makes cryoing free up slots for non-unique jobs #18145

Merged

Conversation

TobleroneSwordfish
Copy link
Contributor

[BALANCE] [RESPAWNING]

About the PR

Marks some jobs (heads of staff) as unique, cryoing as a non-unique job will now immediately free up the slot instead of requiring ghosting. Cryo-ghosting will still free up a unique job slot.

Why's this needed?

Feedback from the #18096 testmerge, people were having issues with secoffs cryoing and still taking up slots. Yes this allows for circumventing slot restrictions by cryoing, but this was already possible under the old system by dying and being cloned so 馃し

Changelog

(u)LeahTheTech
(+)Cryoing as a non-unique job will now immediately free up the slot.

@keywordlabeler keywordlabeler bot added A-Respawning Deals with (re)spawning of mobs. Roundstart, latejoin, events... C-Balance Balance changes, buffs and nerfs labels Mar 8, 2024
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 8, 2024
@goonstation-issuebot goonstation-issuebot added the S-Testmerged [Dev Only] Testmerged for extended testing (applied by bot) label Mar 8, 2024
@Chatauscours
Copy link
Contributor

Chatauscours commented Mar 11, 2024

After playing and observing a few rounds with this I feel that while this is a great change, more jobs should be counted as unique since ending up with two chaplains and two detectives was feeling kinda meh for them from what I gathered.

Imo all command jobs, detective, chaplain, and probably most 1 slot gimmick roles like radio host and inspector should be counted as unique.

@goonstation-issuebot goonstation-issuebot removed the S-Testmerged [Dev Only] Testmerged for extended testing (applied by bot) label Mar 12, 2024
@TobleroneSwordfish TobleroneSwordfish merged commit fad7b01 into goonstation:master Mar 12, 2024
25 checks passed
github-actions bot pushed a commit that referenced this pull request Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Respawning Deals with (re)spawning of mobs. Roundstart, latejoin, events... C-Balance Balance changes, buffs and nerfs size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants