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

The Bartender can now make Eggnog #34065

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Conversation

uhbg
Copy link

@uhbg uhbg commented Dec 25, 2024

About the PR

The Bartender can now make Eggnog. It comes in a wonderful egg shaped glass.

Why / Balance

I wanted to add eggnog.

Technical details

Added eggnog recipe (drinks.yml, alcohol.yml, and other drinks.yml), eggnog sprite (eggnogglass.rsi), and eggnog descriptors (flavors.yml, flavor-profiles.ftl, and alcohol.ftl). Eggnog recipe is 1u egg, 4u milk, 5u rum, all shaken. This produces 10u of eggnog. Eggnog glass sprite has four fill stages. NOTE: Copyright info may be missing from meta.json, wasn't sure if it was needed for this.

Media

eggnogss14

Requirements

Breaking changes

Changelog

🆑

  • add: The Bartender can now make Eggnog by shaking an egg, milk, and rum.

@github-actions github-actions bot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S Denotes a PR that changes 10-99 lines. Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. labels Dec 25, 2024
Copy link
Contributor

github-actions bot commented Dec 25, 2024

RSI Diff Bot; head commit 4290cc3 merging into 7803254
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Objects/Consumable/Drinks/eggnogglass.rsi

State Old New Status
fill-1 Added
fill-2 Added
fill-3 Added
fill-4 Added
icon Added
icon_empty Added

Edit: diff updated after 4290cc3

@slarticodefast slarticodefast self-assigned this Dec 25, 2024
@lzk228
Copy link
Contributor

lzk228 commented Dec 25, 2024

i suggest to add the recipe to the cl

@uhbg
Copy link
Author

uhbg commented Dec 25, 2024

I don't know what caused that test to fail now. It doesn't like non fizzy eggnog.

@ArtisticRoomba
Copy link
Contributor

heisentest fail, someone has to rerun tests

@Tayrtahn
Copy link
Member

Yep, that's #32248 again. Rerunning tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item. Changes: Sprites Changes: Might require knowledge of spriting or visual design. S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. size/S Denotes a PR that changes 10-99 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants