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

Adds ERP #395

Merged
merged 11 commits into from
Aug 23, 2024
Merged

Adds ERP #395

merged 11 commits into from
Aug 23, 2024

Conversation

BonniePandora
Copy link
Contributor

@BonniePandora BonniePandora commented Aug 20, 2024

About the pull request

And by ERP, I mean Eating RolePlay.

Adds various powdered drinks and such to the code. Juices are added from the get-go to MRE packs, with the milk & egg powders getting added once #388 is merged.
Adds a more lore-friendly way of allowing marines to cook without having to rely on some 'magical hypersleep linked freezer' that the galley PR currently uses in the interim.

Explain why it's good for the game

MRE's in modern times come with drink packs already, the current ones in game have a bottle of water, it's frankly wild that no one thought to add juice-powder packs beforehand so those oft ignored bottles of bland water could be spiced up and actually used. Also opens up a new, albiet small, avenue for RP of swapping flavours with other marines and such.
Not to mention the milk & egg powder portion of the PR, which fits more in line with established lore of perishables not being ideal to keep aboard ships travelling in FTL.

Testing Photographs and Procedure

Tested extensively on a local server, no issues found with the code.

Changelog

🆑
add: Added various condiment packets of powedered juices, milk & egg to the code
add: Added corresponding reactions for the new powdered substances where needed
add: Sprites for the condiment packets, some taken from Bay, others edited by me from Bay sprites
qol: MRE's now contain a juice packet alongside their typical contents
maptweak: Replaces the magic freezer and it's perishable contents with a locker full of the non-perishable replacements
/:cl:

Copy link
Contributor

@Doubleumc Doubleumc left a comment

Choose a reason for hiding this comment

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

Code's good, looks like you're just waiting on a sprite for the packets.

Doubleumc
Doubleumc previously approved these changes Aug 22, 2024
@@ -86,7 +86,7 @@
to_chat(user, SPAN_NOTICE(" You transfer [trans] units of the condiment to [target]."))

/obj/item/reagent_container/food/condiment/on_reagent_change()
if(icon_state == "saltshakersmall" || icon_state == "peppermillsmall" || icon_state == "hotsauce_cholula" || icon_state == "hotsauce_franks" || icon_state == "hotsauce_sriracha" || icon_state == "hotsauce_tabasco" || icon_state == "coldsauce_cole")
if(icon_state == "saltshakersmall" || icon_state == "peppermillsmall" || icon_state == "hotsauce_cholula" || icon_state == "hotsauce_franks" || icon_state == "hotsauce_sriracha" || icon_state == "hotsauce_tabasco" || icon_state == "coldsauce_cole" || icon_state == "eggpowder" || icon_state == "milkpowder" || icon_state == "bjpowder" || icon_state == "ojpowder" || icon_state == "ajpowder" || icon_state == "wjpowder" || icon_state == "gjpowder" || icon_state == "pjpowder")
Copy link
Contributor

Choose a reason for hiding this comment

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

this is a fucked up piece of code

. = ..()
reagents.add_reagent("dehydrated_pineapple_juice", 6)

/obj/item/reagent_container/food/condiment/juice/egg
Copy link
Contributor

Choose a reason for hiding this comment

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

mmm egg juice

@AndroBetel AndroBetel merged commit 03b5639 into cmss13-devs:master Aug 23, 2024
35 checks passed
cm13-github added a commit that referenced this pull request Aug 23, 2024
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