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

Rewrite cannabis essential oil extraction #56703

Merged
merged 2 commits into from
Apr 22, 2022
Merged

Rewrite cannabis essential oil extraction #56703

merged 2 commits into from
Apr 22, 2022

Conversation

AtomicFox556
Copy link
Contributor

Summary

Content "Make cannabis essential oil extraction less awkward to perform"

Purpose of change

Currently cannabis oil extraction is extremely awkward to do because it requires making use of tools that need to be activated to be used for crafting; they will be using up batteries during crafting time, and the craft will be interrupted if the battery if fully drained. Also, the processes make use of multiple steps as separate recipes, making this even worse.

Describe the solution

Change the recipes to no longer require active tools, and consolidate all the recipes into all the possible production chains given all the equipment that is currently available. Did some other minor consistency checks to items and recipes.
Removed a "dab pen" item; as much as I was able to figure out, it's essentially the same as electronic cigarette (or at least they could substitute each other), so it was migrated to that.

Describe alternatives you've considered

Getting rid of consumable plastic gaskets as well until there's some better way to simulate parts of machinery wearing down with use.
Making it possible to place all the machinery as appliances, but decided to not do it in this PR as it's already large.

Testing

Launched the game, made sure that no errors happen and that all the recipes work fine.

Additional context

None.

@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Crafting / Construction / Recipes Includes: Uncrafting / Disassembling json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions labels Apr 9, 2022
@github-actions github-actions bot added the BasicBuildPassed This PR builds correctly, label assigned by github actions label Apr 21, 2022
@I-am-Erk
Copy link
Member

Did you make sure that the old item IDs are properly obsoleted or migrated? I'm not seeing them in there, but this is large and I'm on my phone

@AtomicFox556
Copy link
Contributor Author

Did you make sure that the old item IDs are properly obsoleted or migrated? I'm not seeing them in there, but this is large and I'm on my phone

Yes, I did check that I've added obsoletion and migration entries to all the removed recipes and items. I don't think I've missed anything.

@I-am-Erk I-am-Erk merged commit 5c02810 into CleverRaven:master Apr 22, 2022
@Night-Pryanik
Copy link
Contributor

@AtomicFox556 any chance this fixes #43453?

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.

4 participants