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

CorrectContraband levels on each type of magboots #30960

Merged
merged 4 commits into from
Sep 4, 2024

Conversation

TyAshley
Copy link
Contributor

@TyAshley TyAshley commented Aug 13, 2024

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

image
image

Requirements

  • I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

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

  • fix: Magboots and Science Magboots are no longer contraband. Advanced Magboots and Blood-Red Magboots now have appropriate contraband types (Grand Theft and Syndie Contraband respectively)

… magboots to have their own contraband settings
@github-actions github-actions bot added the Changes: No C# Changes: Requires no C# knowledge to review or fix this item. label Aug 13, 2024
@TyAshley TyAshley requested a review from EmoGarbage404 August 13, 2024 04:15
@github-actions github-actions bot added the S: Needs Review Status: Requires additional reviews before being fully accepted label Aug 13, 2024
Copy link
Contributor

@Brandon-Huu Brandon-Huu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TyAshley
Copy link
Contributor Author

@TyAshley TyAshley requested a review from Brandon-Huu August 14, 2024 14:14
@@ -36,7 +37,11 @@
- WhitelistChameleon

- type: entity
parent: [ClothingShoesBootsMag, BaseGrandTheftContraband]
parent: [ClothingShoesBootsMagBase, BaseEngineeringContraband]
Copy link
Contributor

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?

Copy link
Contributor Author

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).

@Emisse
Copy link
Contributor

Emisse commented Aug 19, 2024

only thing contraband should be blood-red magboots

@UbaserB
Copy link
Member

UbaserB commented Aug 28, 2024

You coming back to this?

@TyAshley
Copy link
Contributor Author

TyAshley commented Aug 28, 2024

I've been on vacation and will get back Tuesday and plan on doing it then (unless someone else gets to it first)

@UbaserB
Copy link
Member

UbaserB commented Aug 28, 2024

I've been in vacation an 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

@UbaserB UbaserB mentioned this pull request Aug 28, 2024
2 tasks
…Advanced Magboots are contraband (Grand Theft Contraband). Blood Reds are Syndie Contraband.
@TyAshley
Copy link
Contributor Author

TyAshley commented Sep 4, 2024

I've been in vacation an 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

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).

@UbaserB UbaserB merged commit 4f1518c into space-wizards:master Sep 4, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: No C# Changes: Requires no C# knowledge to review or fix this item. S: Needs Review Status: Requires additional reviews before being fully accepted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bloodred magboots are engi-restricted instead of syndicate contra
5 participants