-
Notifications
You must be signed in to change notification settings - Fork 622
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
Batch lathe jobs #2798
Batch lathe jobs #2798
Conversation
One more thing with the issue here: |
My only fear is that a qol improvement like this one will simply lead to more qol demands in the future. But seriously this is a grand addition. Tested making batch medkits and it was really nice and easy to take a look at what orders I was putting together. |
…-station-14 into 2025-01-28-batch-lathe-jobs
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine, code looks sane, no complaints. :)
About the PR
Items in a lathe's queue now represent some number of the same item to be printed multiple times. This is useful when more than one item is requested (can be done easily with any lathe, since the Amount field is available on all of them.
If nothing else, this reduces the size of the queue which is sent in the lathe BUI state updates in pain points (many many items of the same type).
Why / Balance
Fixes #2792.
How to test
Media
Requirements
Breaking changes
Changelog
🆑