-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
CorrectContraband levels on each type of magboots #30960
CorrectContraband levels on each type of magboots #30960
Conversation
… magboots to have their own contraband settings
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed the base entity to be abstract |
@@ -36,7 +37,11 @@ | |||
- WhitelistChameleon | |||
|
|||
- type: entity | |||
parent: [ClothingShoesBootsMag, BaseGrandTheftContraband] | |||
parent: [ClothingShoesBootsMagBase, BaseEngineeringContraband] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do magboots even qualify as engineering contraband to begin with? They're available in quite a few other places like salv and general EVA?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If magboots are not supposed to be engineering contraband, that is something that can be changed easily. Currently, every type of magboots is listed as engineering contraband. The PR was to fix the egregious issues (namely, bloodreds not being syndicate contraband).
only thing contraband should be blood-red magboots |
You coming back to this? |
I've been on vacation and will get back Tuesday and plan on doing it then (unless someone else gets to it first) |
There's a draft PR open for the bloodred magbots but not the rest so it seems fine to me to just wait for the most part |
…Advanced Magboots are contraband (Grand Theft Contraband). Blood Reds are Syndie Contraband.
I removed engineering contraband from the regular Magboots. Now the only contraband magboots are the CE's Advanced Mag Boots (Grand Theft Contraband) and the Blood-Reds (Syndie Contraband). |
About the PR
All magboots, even blood-reds, say they are engineering contraband. I corrected each type of magboots to reflect the appropriate contraband type. Advanced Magboots are Grand Theft Contraband, as they are a Syndie Objective. Blood Reds are Syndie Contraband. No other magboots are contraband.
Fixes #30931
Why / Balance
Show the correct contraband type to a user when examining an item.
Technical details
Added a ClothingShoesBootsMagBase that is the parent of all other magboot types, this has no contraband level. Modified each of the other magboot child entities to have the appropriate contraband level.
Media
Requirements
Breaking changes
Modified magboots.yml to have a new base/parent entity "ClothingShoesBootsMagBase." This has no contraband parent, and is the parent of all the other magboots types, which now have their own contraband parents set.
Changelog
🆑 Allen