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

[MIRROR] [Bugfix] Worn lolipops properly consume reagents #2314

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

Steals-The-PRs
Copy link
Collaborator

Original PR: NovaSector/NovaSector#1334

About The Pull Request

The lolipops now properly lose reagents when worn in the face slot instead of infinitely giving you its reagents (leading to you inevitably gaining weight.)

This was done by adding a second argument to an AddElement() on lolipops that was missing.

Made primarily because someone requested it in feature requests. I will not lose sleep if this is deemed unnecessary
The other reason was learning a bit of how objects are declared in BYOND, which is a success all on its own.

How This Contributes To The Nova Sector Roleplay Experience

Allows you to cosmetically wear a lolipop for entire shifts without "becoming fat" from the endless stream of nutrient that lolipops could give you. Also the absurdity that it gave infinite omnizine is no more.

Proof of Testing

Screenshots/Videos

I really dont know how to show this, so have a 250% sped up clip of me refreshing the reagents variable till it empites. (clip unedited is ~140ish seconds.
https://github.com/NovaSector/NovaSector/assets/161195966/21eddea2-0cb6-4733-a395-58fcd8abbfcb

Changelog

🆑 LordVoidron
fix: Worn lolipops now properly consume their reagents
/:cl:

Update sweets.dm

The lolipops now properly lose reagents when worn in the face slot instead of infinitely giving you its reagents (leading to you inevitably gaining weight.
@ReezeBL ReezeBL merged commit 0aea856 into master Mar 8, 2024
24 checks passed
@ReezeBL ReezeBL deleted the upstream-mirror-1334 branch March 8, 2024 11:27
AnywayFarus added a commit that referenced this pull request Mar 8, 2024
Iajret pushed a commit that referenced this pull request May 9, 2024
… on languages (for species without name lists, like Felinids and Podpeople) (#2314)

* Random Name Generation refactor, generate random names based on languages (for species without name lists, like Felinids and Podpeople)

* Modular adjustments (vox, teshari names)

* Update yangyu.dm

* Delete language.dm

* Remove language.dm override

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com>
ReezeBL added a commit that referenced this pull request May 11, 2024
… on languages (for species without name lists, like Felinids and Podpeople) (#3268)

* [MIRROR] Random Name Generation refactor, generate random names based on languages (for species without name lists, like Felinids and Podpeople) (#2314)

* Random Name Generation refactor, generate random names based on languages (for species without name lists, like Felinids and Podpeople)

* Modular adjustments (vox, teshari names)

* Update yangyu.dm

* Delete language.dm

* Remove language.dm override

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com>

* now it compiles.

---------

Co-authored-by: NovaBot <154629622+NovaBot13@users.noreply.github.com>
Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: Bloop <13398309+vinylspiders@users.noreply.github.com>
Co-authored-by: ReezeBL <shilk.e82@gmail.com>
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