Skip to content

Smith Polish: Adds access requirements to smith machines #29295

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

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

Conversation

PollardTheDragon
Copy link
Contributor

@PollardTheDragon PollardTheDragon commented May 11, 2025

What Does This PR Do

Locks smith machines (casting basin, power hammer, lava furnace, and kinetic assembler) to ACCESS_SMITH or ACCESS_MINING. The Scientific Assembler requires either ACCESS_TOX or ACCESS_XENOBIOLOGY. An emag can be used to short out these requirements.

Why It's Good For The Game

Generally, the intention behind the smith is for the departments to have their own machines to incentivize working together, not having one department build a monolith. This helps enforce that.

Testing

Tried to use smith machines as scientist. Could not. Was able to as the smith. Emagged the machine and could then use it as the scientist.

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

🆑
tweak: Smith machines now are access locked.
/:cl:

@ParadiseSS13-Bot ParadiseSS13-Bot added -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 -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 May 11, 2025
@ParadiseSS13-Bot ParadiseSS13-Bot added -Status: Awaiting merge This PR is ready for merge and removed -Status: Awaiting review This PR is awaiting review from the review team labels May 15, 2025
@github-actions github-actions bot added the Merge Conflict This PR is merge conflicted label May 17, 2025
@github-actions github-actions bot removed the Merge Conflict This PR is merge conflicted label May 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-Status: Awaiting merge This PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants