8000 Satchels only auto-fill with produce from harvests, ignore seeds by Egregorious · Pull Request #23745 · goonstation/goonstation · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Satchels only auto-fill with produce from harvests, ignore seeds #23745

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

Conversation

Egregorious
Copy link
Contributor

About the PR

Produce satchels no longer have a popup option to harvest seeds or produce. Using them on a harvestable plant will instead automatically harvest and place produce in satchel and dump the seed packet on the floor.

Why's this needed?

Even before seed packets, I found the option to harvest seeds directly into the satchel was very rarely used. The mechanic feels practically obsolete now that there aren't multiple seed items per harvest. The popup is annoying and is entirely counter-productive to the qol that auto-filling satchels directly from harvests intends to confer.

Testing

I'm not sure screenshots are useful for this because it's like trying to prove a negative.

Tested multiple harvests with an empty satchel; until satchel became full, after satchel was full, before plant was ready to harvest and after plant had died.
Screenshot 2025-06-05 080451

Changelog

(u)Egregorious
(+)Harvesting plants with a produce satchel now auto-fills it with produce directly and ignores the seed packet, instead of demanding input.

@keywordlabeler keywordlabeler bot added A-Hydroponics Deals with plants, hydro, etc. C-QoL A quality of life improvement that makes the game easier to play labels Jun 5, 2025
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 5, 2025
@NibChocolateeny
Copy link
Contributor

it is interesting idea...satchel still can hold seed, so good with large seed transporting. worth trying!

@Egregorious Egregorious changed the title Satchels only auto-harvest produce from harvests, ignore seeds Satchels only auto-fill with produce from harvests, ignore seeds Jun 5, 2025
@Scaltra-Volpe
Copy link
Contributor

Personally when doing very large bulk producing i find the seeds are really annoying and I wish I could specifically get the seeds and only the seeds without a pop up menu.

Personally i would prefer to be able to still grab seeds directly with the satchel with some way to switch which is done (either way I would still even like this version because honestly its faster to handclick and clicl drag collect the seeds afterward anyway)

Copy link
Contributor
@TobleroneSwordfish TobleroneSwordfish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this makes sense, the menu has always been annoying.

@Egregorious
Copy link
Contributor Author

Personally i would prefer to be able to still grab seeds directly with the satchel with some way to switch which is done

I didn't bother preserving it because I didn't think anyone used the seed-only collection option honestly, but if people do then it's probably worth preserving.

I think this is doable with intent-specific interactions. I'll make a different PR with an attempt at it, since it'll be rather different than this implementation.

Copy link
Member
@frawhst frawhst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer this to #23750, we try to shy away from intent interactions on objects and overall I think the QoL gained is worth the minor functionality lost (and at worst you just need to click-drag a seed into the satchel)

@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Jun 5, 2025
@TobleroneSwordfish TobleroneSwordfish merged commit 543d494 into goonstation:master Jun 6, 2025
28 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 6, 2025
@Egregorious Egregorious deleted the Produce-satchels-no-take-seeds branch June 14, 2025 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Hydroponics Deals with plants, hydro, etc. C-QoL A quality of life improvement that makes the game easier to play S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0