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

Dynamic trees re-compat #9706

Open
wants to merge 1 commit into
base: version/main
Choose a base branch
from
Open

Conversation

uecasm
Copy link
Contributor

@uecasm uecasm commented Jan 14, 2024

Closes discord request
Closes discord request

Changes proposed in this pull request:

  • Enable Dynamic Trees compatibility for 1.20

[x] Yes I tested this before submitting it.
[ ] I also did a multiplayer test.

Review please (do not backport)

Might want to avoid merging until DynamicTreesTeam/DynamicTrees#835 is fixed; it didn't crash on me when the forester was felling the tree, but then nobody was standing in the way...

@uecasm uecasm marked this pull request as draft January 14, 2024 09:26
@uecasm
Copy link
Contributor Author

uecasm commented Jan 14, 2024

Switching to draft due to maven issues.

marchermans
marchermans previously approved these changes Jan 14, 2024
@uecasm uecasm force-pushed the feature/dynamic-trees-120 branch from fa759a2 to c1c92db Compare July 2, 2024 08:28
@uecasm uecasm marked this pull request as ready for review July 2, 2024 08:29
@uecasm
Copy link
Contributor Author

uecasm commented Jul 2, 2024

It lives again!

@supermassimo
Copy link

Adding to this, the latest beta of DT now includes a "falling_tree_damage_immune" entity type tag. If the colonist entity is added to this tag they will not be dealt damage by falling trees

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants