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

[MIRROR] issue fixes and qol for ant, worm, and ash farming #269

Merged
merged 1 commit into from
Oct 25, 2023

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: Skyrat-SS13/Skyrat-tg#24543

About The Pull Request

you can now feed worm and ant farms from a plant bag
plant bags can now directly harvest from ash farms
worms will reproduce when having 2 (was bugged and required 3 due to logic)
fertilizer will not be used if it doesn't upgrade something in the plant

How This Contributes To The Skyrat Roleplay Experience

there are a few things that didn't make sense-- worms can't reproduce when having 2? it was a logic error: using > instead of >=. there are also a few qol things that should have been added way earlier

Proof of Testing

Screenshots/Videos

image
image

Changelog

🆑 jjpark-kb
qol: you can now directly feed worm/ant farms from plant bags
qol: you can now directly harvest ash farms with plant bags
fix: worm farms can reproduce with 2 now (instead of the 3 due to a logic error)
fix: fixed fertilizer being deleted even if it didn't upgrade an ash farm
/:cl:

* fixes some issues with ash, worm, and ant farming + qol

* Update modular_skyrat/modules/ashwalkers/code/buildings/wormfarm.dm

Co-authored-by: Bloop <[email protected]>

---------

Co-authored-by: Bloop <[email protected]>
@ReezeBL ReezeBL merged commit 04e7947 into master Oct 25, 2023
23 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-24543 branch October 25, 2023 06:25
AnywayFarus added a commit that referenced this pull request Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants