Skip to content

Conversation

RoyalFox2140
Copy link
Collaborator

@RoyalFox2140 RoyalFox2140 commented Apr 16, 2023

SUMMARY: Balance "Ported the Broken Generator deconstruct from DDA"

Purpose of change

The broken generator is one of few furnitures/furniture terrains that have no deconstruct recipe. This ports the DDA deconstruct.

Describe the solution

I copied their deconstruct recipe but replaced pipe fittings with 0-1 scrap metal as this item does not exist in BN.

Describe alternatives you've considered

Changing the resulting items so it doesn't give a whole plastic jerrycan. Leaving it as smashing required.

Testing

I opened the game with the changes and successfully deconstructed a broken generator for parts. No crashes or errors.

Additional context

If I run into any other common furnitures that isn't deconstructible but should, I'll be sure to tackle them.

broken generator deconstructible, changed pipe fittings to scrap metal as it doesn't exist.
@github-actions github-actions bot added the JSON related to game datas in JSON format. label Apr 16, 2023
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

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

could you please link the ported DDA PR and author? (in the PR body)

@RoyalFox2140
Copy link
Collaborator Author

Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

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

preview deconstruction
Cataclysm: Bright Nights - cbn-experimental-2023-04-15-1839_01 Cataclysm: Bright Nights - cbn-experimental-2023-04-15-1839_02

LGTM

@scarf005 scarf005 merged commit 0014ccd into cataclysmbnteam:upload Apr 16, 2023
@RoyalFox2140 RoyalFox2140 deleted the broken-generator-deconstructible branch June 6, 2023 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants