Skip to content

Modular Bsaecamp Workshop recipe update#60582

Merged
dseguin merged 1 commit intoCleverRaven:masterfrom
Sathra225:modular-basecamp-qualitysteel
Sep 1, 2022
Merged

Modular Bsaecamp Workshop recipe update#60582
dseguin merged 1 commit intoCleverRaven:masterfrom
Sathra225:modular-basecamp-qualitysteel

Conversation

@Sathra225
Copy link
Contributor

@Sathra225 Sathra225 commented Aug 30, 2022

quality steel and steel products for modular workshop

Summary

Content "Update modular basecamp workshop 2 recipes"

Purpose of change

#59682 added the different steel quality lumps and wire and such to the basecamps, but the modular basecamp workshop 2 uses its own recipe set that was missed. Adding the recipes to that list as well (with roughly the same progression but the modular isn't 1 to 1).

Describe the solution

Copy-pasting the new recipe fields into the modular basecamp workshop 2 recipe groups.

Describe alternatives you've considered

None.

Testing

Made changes, loaded game with a modular basecamp workshop 2 and verified that the recipes were available.

Additional context

quality steel and steel products for modular workshop
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling labels Aug 30, 2022
@Sathra225
Copy link
Contributor Author

Huh, the modular version 2 didn't have glass caltrops so those those got added too.

@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Aug 30, 2022
@PatrikLundell
Copy link
Contributor

The reason the workshop 2 recipe lists was kept short is that the UI works very poorly when the lists are bloated. The glass caltrops were not introduced since I didn't think they'd be sufficiently useful for companion construction to be worth a place on the list. However, I've been overruled on the bloat issue in the past, and so expects to be so again (The argument was approximately that nobody will fix the bloat issue until it becomes painful. My counter argument is that it's been painful for a fair while, but nobody has overhauled the UI yet despite that).

@Sathra225
Copy link
Contributor Author

Sathra225 commented Aug 30, 2022

True, though this PR is largely so the recipe lists between the two 'versions' of basecamp workshops are the same.
Or close to the same. The other recipe list also has the massive wall of variant plate so they're now equally problematic.

@dseguin dseguin merged commit dfb72d0 into CleverRaven:master Sep 1, 2022
@Sathra225 Sathra225 deleted the modular-basecamp-qualitysteel branch September 1, 2022 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions Crafting / Construction / Recipes Includes: Uncrafting / Disassembling [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants