Allergies (again. I hate Database changes.)#307
Open
ScarlettJFG wants to merge 4 commits intoFloof-Station:masterfrom
Open
Allergies (again. I hate Database changes.)#307ScarlettJFG wants to merge 4 commits intoFloof-Station:masterfrom
ScarlettJFG wants to merge 4 commits intoFloof-Station:masterfrom
Conversation
Contributor
Author
|
Please, whatever god is listening, let this work. |
|
I may not be god, but it looks like its gonna work |
Contributor
|
So it looks like those SQL errors are legitimate. You may want to go through what Mnemo said and see if that fixes things. |
|
We're pretty sure that databases were invented in hell to punish contributors. |
|
Also checking the comments of the original PR, this was mentioned: cosmatic-drift-14/cosmatic-drift#649 (comment) It may be part of the solution as they mentioned databases in this PR that comment is referencing: cosmatic-drift-14/cosmatic-drift#727 (comment) |
Mnemotechnician
requested changes
Mar 23, 2026
Collaborator
Mnemotechnician
left a comment
There was a problem hiding this comment.
Missing database migrations, errors might or might not be related
# Conflicts: # Content.Client/Lobby/UI/HumanoidProfileEditor.xaml.cs # Content.Server.Database/CDModel.cs # Content.Server.Database/Model.cs # Content.Server/Database/ServerDbBase.cs # Content.Shared/Preferences/HumanoidCharacterProfile.cs
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
About the PR
Adds Allergies and the associated effects to the game.
Why / Balance
EVERYONE WANTS MEDICAL TO [SUFFER](https://discord.com/channels/1255902263667331193/1478151943078674484/1482892877188173925)Technical details
I'll be completely honest: It's a near 1:1 port from this PR, but with a minor change (I...didn't need to add lines for the allergies to grab what was processed). This adds changes to the server.database folder for records, HumanoidCharacterProfile for attaching the allergies, Histamine in the toxins.yml, and frankly, this is just a port. All of the edits however, allows the selections from the profile editor in lobby to be used, but does not sync the "Allergies" text field, with what is selected. Medical should be able to pick up on an allergen fairly quickly however, so this doesn't...completely screw them over.
Asked to Reupload PR to see if this fixes the Database errors that were present in the previous PR.
Media
Click to show
Discord link to Video (Will open in browser, a little too big to upload here)
Requirements
Licensing:
Breaking changes
Allergies selected do not update in the Allergies text field in records.
Transforming (Geras or fox form) will temporarily pause allergic reactions, at least until transformed back. This can be used as a way to treat certain issues you might otherwise be allergic to, without death...unless it's too late.
Sources
As listed above, This PR is where the port is from.
Changelog
🆑