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 monster hunter blood vial is now just a status effect instead of a weird status effect + reagent combination #4578

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

Absolucy
Copy link
Member

@Absolucy Absolucy commented Dec 15, 2024

About The Pull Request

this changes the MH blood vial to just be implemented in /datum/status_effect/cursed_blood, rather than a weird split between /datum/status_effect/cursed_blood and /datum/reagent/medicine/blood_vial

i've moved the status effect to the priority processing speed, as it's something where delayed ticks can be the difference between life and death.

also gave it an icon, albeit just the blooddrunk icon, and slightly cleaned up the code overall.

Why It's Good For The Game

it being a reagent kinda fucks over IPC monster hunters and such, and it's kinda dumb anyways that it was both a status effect AND a reagent. also it was possible to be allergic to the blood vial, which was... unfortunate if someone with allergies actually rolled MH.

so let's just make it a status effect.

Changelog

🆑
refactor: The effect from a monster hunter's blood vial is now just a status effect, rather than a weird combination between a status effect and a reagent.
/:cl:

@Absolucy
Copy link
Member Author

need to do more testing and tweaking to ensure that the effects are close enough or the same as previously

@Absolucy Absolucy force-pushed the mh-blood-thingy-fixup branch from 1762f48 to f91595d Compare December 20, 2024 06:19
@ThePooba ThePooba merged commit 1382410 into Monkestation:master Dec 20, 2024
24 checks passed
github-actions bot added a commit that referenced this pull request Dec 20, 2024
@Absolucy Absolucy deleted the mh-blood-thingy-fixup branch January 7, 2025 21:08
Gw0sty pushed a commit to Gw0sty/Monkestation2.0 that referenced this pull request Jan 9, 2025
Gw0sty pushed a commit to Gw0sty/Monkestation2.0 that referenced this pull request Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do not merge don't merge this ffs Minor Refactor/Rework Completely redoing a larger thing / recoding a thing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants