Cleaning up Borg's construction Module #1095

Open
Balenar wants to merge 2 commits from Balenar/Byrd:ConstructionModuleImprovements-10202026 into master
Contributor

Short description

Replacing the pile of hard locked slots in the construction module with a couple of droppable slots that can hold any floor tile and any material

Why we need to add this

it is a PAIN as a engiborg to try to interact with floors or actually build anything that uses a slightly non-standard material like brass, silver, or gold, giving them the ability to hold any material or floor tile should help with that immensely without majorly changing balance

Checklist

Ingame Changelog

🆑

  • tweak: Cleaned up construction module, it now uses 3 whitelisted slots rather than 8 locked slots
<!-- Link definitions, do not edit --> [guidelines]: https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html [ruleset]: https://rules.byrdstation.space/#h.vigrxrnffwji ## Short description <!-- What do you propose to change with your PR? --> Replacing the pile of hard locked slots in the construction module with a couple of droppable slots that can hold any floor tile and any material ## Why we need to add this <!-- What is the reason for adding these changes? Please post links to Discussions as well as Bug Reports here. Please describe how this will change the game balance. --> it is a PAIN as a engiborg to try to interact with floors or actually build anything that uses a slightly non-standard material like brass, silver, or gold, giving them the ability to hold any material or floor tile should help with that immensely without majorly changing balance ## Checklist <!-- Check boxes for faster reviewing of your PR --> - [X] I do not require assistance to complete the PR. - [X] I have verified that the changes work before posting/requesting review of my PR. - [X] I have added screenshots/videos of the changes, or this PR does not change in-game mechanics. - [X] I have read the [Pull Request and Changelog Guidelines][guidelines] and [Byrd Station's Contributor Ruleset][ruleset]. ### Ingame Changelog <!--If you want the players to know about changes made in this PR, specify them using the template outside the comment. Short and informative.--> :cl: - tweak: Cleaned up construction module, it now uses 3 whitelisted slots rather than 8 locked slots
Replacing construction's mess of item slots with droppable item slots for materials and floortiles
Some checks failed
Close PRs on master / run (pull_request_target) Has been skipped
Labels: PR / labeler (pull_request_target) Has been cancelled
Labels: Size / size-label (pull_request_target) Has been cancelled
CRLF Check / CRLF Check (pull_request) Has been cancelled
Test Packaging / Test Packaging (pull_request) Has been cancelled
RGA schema validator / YAML RGA schema validator (pull_request) Has been cancelled
RSI Validator / Validate RSIs (pull_request) Has been cancelled
Map file schema validator / YAML map schema validator (pull_request) Has been cancelled
YAML Linter / YAML Linter (pull_request) Has been cancelled
8652d215ff
Gave them a extra material slot for a total of 2
All checks were successful
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Successful in 2s
CRLF Check / CRLF Check (pull_request) Successful in 32s
Test Packaging / Test Packaging (pull_request) Successful in 12m54s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 38s
RSI Validator / Validate RSIs (pull_request) Successful in 3m19s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m56s
YAML Linter / YAML Linter (pull_request) Successful in 4m31s
0f80bbc1fe
All checks were successful
Labels: PR / labeler (pull_request_target) Successful in 5s
Labels: Size / size-label (pull_request_target) Successful in 2s
CRLF Check / CRLF Check (pull_request) Successful in 32s
Test Packaging / Test Packaging (pull_request) Successful in 12m54s
RGA schema validator / YAML RGA schema validator (pull_request) Successful in 38s
RSI Validator / Validate RSIs (pull_request) Successful in 3m19s
Map file schema validator / YAML map schema validator (pull_request) Successful in 5m56s
YAML Linter / YAML Linter (pull_request) Successful in 4m31s
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u "ConstructionModuleImprovements-10202026:Balenar-ConstructionModuleImprovements-10202026"
git switch "Balenar-ConstructionModuleImprovements-10202026"
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Byrd-Station/Byrd!1095
No description provided.