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

Adds job overalls to the loadout menu #27427

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Screampuff
Copy link

@Screampuff Screampuff commented Nov 22, 2024

What Does This PR Do

Add the job overalls from the various vendors to the player's loadout.

Why It's Good For The Game

They're good overalls, and not having to buy them and then police strip yourself every round start just to get them would be nice!

Images of changes

image
image

Testing

Ran game, checked if overalls were there and selected it as Janitor, spawned in with overalls.


Declaration

  • I confirm that I either do not require pre-approval for this PR, or I have obtained such approval and have included a screenshot to demonstrate this below.

image

Changelog

🆑
add: Added job overalls to the player loadout
🆑

@ParadiseSS13-Bot ParadiseSS13-Bot added the -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally label Nov 22, 2024
@Screampuff Screampuff changed the title Adds janitor's overalls to the loadout menu Adds job overalls to the loadout menu Nov 22, 2024
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting approval This PR is waiting for approval internally -Status: Awaiting review This PR is awaiting review from the review team and removed -Status: Awaiting type assignment This PR is waiting for its type to be assigned internally -Status: Awaiting approval This PR is waiting for approval internally labels Nov 22, 2024
Copy link
Contributor

@lewcc lewcc left a comment

Choose a reason for hiding this comment

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

Code mostly looks good, just one nit

code/modules/client/preference/loadout/loadout_uniform.dm Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting review This PR is awaiting review from the review team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants