-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Farming plot zone - QoL improvement features #24826
Comments
Looks sweet, great design idea. |
I have planned to do something like thant in the future, but you already done most of the work. |
In my work on planting, I heavily refactored iexamine::dirtmound to pieces, so I could call the same code. If your watering etc. will be call from the outside ready, it will make the things easier for me. But yeah, no need to do it now. Let's wait until your PR is merged. |
Ideally, I really like when such features are optional. |
Pull request #27278 contains a candidate implementation of zone activity commands for harvesting and fertilization. |
Closing as this is now addressed by zone farming activities, which now includes harvesting, and fertilizing. |
Farming plot zone
Aims to improve farming ease of use and to allow for further farming enhancements without making it less user friendly.
Current state of farming
While farming is a great feature, it tends to be key press heavy and this problem seems to be increasing with time, rather than decreasing. In the near future, an Improve crop farming PR #24291 will be introduced and while it greatly enhances farming, some did express worries about excessive tediousness of such system.
I happen to agree that something has to be done to allow for such great improvement without killing all the fun in farming for the user.
Quality of life features to help remedy the issue
While I do have designating seed and planting marked as done, these would have to be updated, once #24291 is merged.
To give a better idea, how would such system work, I prepared short 1 minute video with my current implementation in action.
https://youtu.be/H8tBR3GdZLE
The text was updated successfully, but these errors were encountered: