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

[Modding] Add GENERIC Structure types #3773

Closed
Arithyce opened this issue Apr 28, 2024 · 0 comments · Fixed by #3777
Closed

[Modding] Add GENERIC Structure types #3773

Arithyce opened this issue Apr 28, 2024 · 0 comments · Fixed by #3777
Labels
Milestone

Comments

@Arithyce
Copy link

Arithyce commented Apr 28, 2024

Describe the feature you'd like
EDIT FOR CLARITY:
Structure is the value used in research.json for upgrading base structures.
Wall is the value used in research.json for upgrading defense structures.
DEFENSE, WALL, WALL CORNER, GATE, and GENERIC are considered Wall for upgrade purposes, everything else is Structure.

There really needs to be an equivalent of GENERIC for structures (Structure), maybe GENERAL or MISC. Right now, if you want to add a new structure with no functionality, so that you can add functionality through a script, you need to use either DEFENSE or GENERIC, but then your structure will be considered a Wall, which is undesirable.
EDIT FOR CLARITY: For the purposes of applying upgrades through research, this is important.

Describe why do you think it is needed
There is currently no way to add new structures (Structure) with new functionality, while you can add new defenses (Wall) with new functionality. There is an exception in the MISSILE SILO, but that has potentially unwanted traits, such as being packable and explosive.

More information
I can easily add this myself, so i will be creating a PR soon with the addition.
Fixed by: #3777

@Arithyce Arithyce changed the title Modding feature request [Modding] Add GENERIC Structure types Apr 29, 2024
@KJeff01 KJeff01 added this to the 4.5.0-beta1 milestone Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants