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.

Sign up for GitHub

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

Jump to bottom

Vehicle Framework - Component depth effected by upgrades #3302

Merged
merged 1 commit into from
Jul 27, 2024
Merged

Vehicle Framework - Component depth effected by upgrades #3302

merged 1 commit into from
Jul 27, 2024

Conversation

Rhinous
Copy link
Contributor

@Rhinous Rhinous commented Jul 25, 2024

Changes

  • Component depth is now effected by upgrades

Reasoning

  • Should mean that everything from VF new upgrade system is working properly now. (Other values already correctly upgrade)

Alternatives

  • N/A

Testing

Check tests you have performed:

  • Compiles without warnings
  • Game runs without errors
  • (For compatibility patches) ...with and without patched mod loaded
  • Playtested a colony - Devtest with the CV90 mod changed to have tracks default as external > after armor plating upgrade they correctly changed to internal

@Rhinous Rhinous requested review from a team as code owners July 25, 2024 06:02
@github-actions GitHub Actions
Copy link

You can download the rebuilt assembly for this PR here: https://combatextended.lp-programming.com/CombatExtended-10088981630.zip

@github-actions github-actions bot added the Download in Comments This PR has a zipfile download available. label Jul 25, 2024
@N7Huntsman N7Huntsman merged commit ddcc4dc into CombatExtended-Continued:Development Jul 27, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@N7Huntsman N7Huntsman N7Huntsman approved these changes

@Thorbane Thorbane Thorbane approved these changes

Assignees
No one assigned
Labels
Download in Comments This PR has a zipfile download available.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Rhinous @Thorbane @N7Huntsman