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

Move tokens from roundstart sec inventories to seclinks #1631

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

Pinkbat5
Copy link

Before seclinks were introduced, anybody that broke into a security locker could nab a disabler or stun baton out of it. Newly deputized officers could just walk in and grab their things from the locker room without assistance from a Warden or HoS, and if you're an antag on a map with one of those isolated security outposts with lockers in them, breaking in would net you some valuable loot. This PR lets you do both again.
While it is now possible for anybody in security to loot multiple lockers and buy two disablers or whatever, it was also possible to grab multiple disablers from lockers before seclinks were introduced, and that didn't happen so much. I suspect most people are not going to be a dick and take too much from the lockers. If they do, the Warden still has spare tokens they can give out.

Changelog

🆑

  • tweak: Baton, disabler and utility tokens now start in seclinks instead of roundstart security inventories.

@Sha-Seng
Copy link

please. we need antags to be rewarded for stealing sec lockers

@hivehum
Copy link

hivehum commented Feb 10, 2025

looks fine to me but im gonna let it be open for a day or so in case anyone wants to discuss

@Darkmajia
Copy link

i'm in favour for what it's worth

Copy link

@hivehum hivehum left a comment

Choose a reason for hiding this comment

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

20 hours no objections and i want to run a publish

@hivehum hivehum merged commit a7d6c24 into impstation:master Feb 10, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants