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

Add security magboots #2074

Merged
merged 6 commits into from
Nov 6, 2024
Merged

Conversation

Radezolid
Copy link
Contributor

@Radezolid Radezolid commented Oct 30, 2024

About the PR

Added the security magboots. The sprites were done by ProphetCrow and used here with their permission, this PR is a remake of their abandoned PR #1490.

Why / Balance

I've seen lube being used by nuclear operatives a lot lately and it leaves almost no counterplay from the crew against the nukies.
These aren't meant for security to wear all the time and should be considered powergaming, since it's the same reason they shouldn't wear EVA suits all the time.

Technical details

Modified the MagnetsTech tech to make the security magboots researcheable, added recipe for the sec techfab to be able to print it them, modified the table for the HOS hardsuit spawn, and added the magboot to the sec suit storages, including warden's locker.

Media

2024-10-30.00-43-01.mp4

Security TechFab:
image

Research:
image

Requirements

Breaking changes

Hopefully none.

Changelog

🆑 Crow, Radezolid

  • add: Added security magboots, they should only be used for EVA or situations that warrant no slips.

@Radezolid Radezolid requested review from IamVelcroboy and a team as code owners October 30, 2024 04:55
@github-actions github-actions bot added Status: Needs Review Changes: YML Changes any yml files Changes: Sprite Changes any png or json in an rsi labels Oct 30, 2024
Copy link
Contributor

github-actions bot commented Oct 30, 2024

RSI Diff Bot; head commit f859fe4 merging into 1e0c4f3
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/DeltaV/Clothing/Shoes/Boots/magboots-security.rsi

State Old New Status
equipped-FEET Added
icon-on Added
icon Added
inhand-left Added
inhand-right Added
on-equipped-FEET Added
on-inhand-left Added
on-inhand-right Added

Edit: diff updated after f859fe4

Copy link
Contributor

github-actions bot commented Nov 3, 2024

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@deltanedas deltanedas added Status: Awaiting Changes Do not merge due to requested changes and removed Status: Needs Review labels Nov 3, 2024
@github-actions github-actions bot added Status: Needs Review and removed Status: Awaiting Changes Do not merge due to requested changes labels Nov 6, 2024
@deltanedas deltanedas enabled auto-merge (squash) November 6, 2024 09:56
@github-actions github-actions bot removed the Status: Merge Conflict Fix your PR! label Nov 6, 2024
@deltanedas deltanedas merged commit 736accd into DeltaV-Station:master Nov 6, 2024
13 checks passed
@Radezolid Radezolid deleted the Sec-magboots branch November 6, 2024 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Sprite Changes any png or json in an rsi Changes: YML Changes any yml files Status: Needs Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants