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

RCD can deconstruct the more common walls/window #1950

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

VividPups
Copy link
Contributor

@VividPups VividPups commented Sep 7, 2024

New wall destructions

About the PR

adds more things to deconstruct on the RCD

Why / Balance

Request and its really annoying spending like 40 mins wacking at a wall just to remove it.
I believe it would be ok since now it require the Captain ID to used the Shipyard RCD

How to test

wall plus RCD together

Media

  • this PR does not require an ingame showcase

Changelog

🆑

  • tweak: RCD ability to deconstruct more walls

New wall destructions
@dvir001
Copy link
Contributor

dvir001 commented Sep 7, 2024

Changelog need to be more clear, missing # Frontier comment on all the added lines

Also note parenting that wont need to be added 2 times.

@dvir001 dvir001 added Status: Awaiting Changes This PR has changes that need to be made before merging Status: DO NOT MERGE labels Sep 7, 2024
@VividPups
Copy link
Contributor Author

Changelog need to be more clear, missing # Frontier comment on all the added lines

Also note parenting that wont need to be added 2 times.

Huh?

frontier tag update
@VividPups
Copy link
Contributor Author

@dvir001 I believe I added all the tags

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No C# Status: Awaiting Changes This PR has changes that need to be made before merging Status: DO NOT MERGE YML
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants