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

DrinkWave3 #121

Merged
merged 11 commits into from
Aug 21, 2024
Merged

DrinkWave3 #121

merged 11 commits into from
Aug 21, 2024

Conversation

SleepyScarecrow
Copy link
Contributor

Description

Adding in 24 more community-suggested drinks.


TODO

  • Added more Drinks

Changelog

🆑 Tilkku

  • add: 24 More Drinks

@github-actions github-actions bot added Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Status: Needs Review Someone please review this labels Aug 19, 2024
Copy link
Contributor

github-actions bot commented Aug 19, 2024

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

Resources/Textures/Floof/Objects/Consumable/Drinks/koboldkooler.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/koboldkorkscrew.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/lamplight.rsi

State Old New Status
fill-1 Modified
fill-2 Modified
fill-3 Modified
icon Modified
icon_empty Modified

Resources/Textures/Floof/Objects/Consumable/Drinks/salvagersdelight.rsi

State Old New Status
fill-1 Modified
fill-2 Modified
fill-3 Modified
icon Modified

Resources/Textures/Floof/Objects/Consumable/Drinks/voltage.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/7deadlysins.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/appleberryfizz.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/bande.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/bethanyscumlust.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/bethanyssluttycumlust.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/cafedecacao.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/calicosunrise.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/cremedecacao.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/darkoasis.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/deadmansdraught.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/dirtywastelander.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/heartbreakmountain.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/holyvice.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/numberone.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/pomonade.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/prisonwine.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/quadruplecreamchalice.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/slimesdelight.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/snowball.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/specialsundae.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/thewaterslide.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/triplecitrus.rsi

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

Resources/Textures/Floof/Objects/Consumable/Drinks/wakeupjuice.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 08169bc

@ShatteredSwords
Copy link
Contributor

Test fails are legit, issues are as follows:

  • DeadMansDrought needs reagent-desc-deadmansdrought Localization
  • Dirty Wastelander needs reagent-physical-desc-dirty Localization
  • BethanysSluttyCumLust needs reagent-desc-BethanysSluttyCumLust

@ShatteredSwords
Copy link
Contributor

Ah lools like you handled em

@ShatteredSwords
Copy link
Contributor

You seem to know what you are doing, I'll leave you to it o7

@SleepyScarecrow
Copy link
Contributor Author

ignore that commit name i fucked up cherry picking lol

@ShatteredSwords
Copy link
Contributor

That error looks valid but I honestly couldn't tell you what's wrong with it

@SleepyScarecrow
Copy link
Contributor Author

I can't figure it out either
I've had this happen a lot, with the random build error that is actually fine but idk

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Aug 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Aug 20, 2024
@SleepyScarecrow
Copy link
Contributor Author

idk if the build fail is valid or not

@SleepyScarecrow
Copy link
Contributor Author

like I've seen this before but it doesn't actually seem to fuck up anything?

@ShatteredSwords
Copy link
Contributor

There is a possibility that one of these newly added drinks overwrites another recipe. Worth taking a look at.

@ShatteredSwords
Copy link
Contributor

There is also a possibility that one of the vanilla recipes overwrites one of these drinks before they can be made.

@SleepyScarecrow
Copy link
Contributor Author

i think the issue was the prisonwine recipe, it directly overwrote the grapesoda recipe

@SleepyScarecrow
Copy link
Contributor Author

works now :3

@FoxxoTrystan FoxxoTrystan merged commit 531379c into Fansana:master Aug 21, 2024
13 checks passed
Floof-Station-Bot added a commit that referenced this pull request Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: Localization Changes any ftl files Changes: Sprite Changes any png or json in an RSI Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants