-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Delivery Packages (part 1) #38155
Delivery Packages (part 1) #38155
Conversation
- changes cardboard boxes to work for packages - adds package items
You need to lint your changes, you can use this http://dev.narc.ro/cataclysm/format.html |
Yeah, that is what I used. But I did reconfigure some of the short lists at the top of /itemgroups/packages.json as they are easier to read vertically than horizontally. I could revert these if need be... |
Automated checks disagree with this logic. |
I've got some boxes I've used in houses and basements, they have general house spawns in them mostly. Be cool to move them into this packaging system eventually. So, I'd suggest household junk boxes, they type of stuff you store and forget about for 20 years, maybe even holiday decorations and such if you want to get fancy. Their spawn list is in: https://github.com/CleverRaven/Cataclysm-DDA/blob/master/data/json/mapgen_palettes/house_general_palette.json if you want to look. symbol for boxes is "z" |
@curstwist I believe I made some item groups of the same fashion somewhere too... We didn't have cardboard boxes at the time so I used crates, and the crates would spawn living room/kitchen/garage/etc stuff depending on what room. They were mostly only used in the 'vacant homes' pack I did which included homes that had garage sales, we're about to move out, just moved in, etc. I agree it would maybe be nice to consolidate all this into the new system at some point in the future. |
ah right, I have seen those. I'll take a look at their groups. |
This is a prime opportunity for a movie reference. Add a |
There doesn't seem to be one in-game. Though in the future it would be easy to add as well as a park location with a sandy volleyball court. |
This is too much for a single PR. Can you keep the list of packages down a bit, and add more packages in another PR? It seems to me packages are something that could use the disassembly script like MREs, rather than just appearing in itemgroups |
Uhm, I guess. The only thing removing some item groups from this PR would do is lower line count. I admit I had some more item groups ready to push and add to this PR but Ill wait and spread things out more if that's needed.
I plan to use the 'unpack' being moved to json in #38111 (which the gobag uses) |
Broke this PR into more pieces as requested by Erk. This first part I believe is ready. |
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.
If folding for smaller total volume is a feature of the box, then the box needs to explicitly have that feature, not have contradictory stats.
Closing this until I have time to come back to it. |
Summary
SUMMARY: Content "Adds delivery packages."
Purpose of change
This adds delivery packages of which can be unpacked to reveal random goodies. It's like Christmas any day of the year!
This is a revival of what I started roughly 1.5 years ago and will be split into several parts.
Packages come in 3 sizes like the cardboard boxes. Their contents are based on their size, and you can find the list of items I made before in the link above. But due to the fact I made the package content items groups 1.5 years ago, I may be missing some obvious candidates for package contents that have been added to the game since then. So, if you have any suggestions of additional package contents, let me know and I can add them. 300+ DIFFERENT PACKAGE ITEM GROUPS!!!
Changes in this pr:
Additional Context
Changes in following prs: