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

Add check_for_pole to some poles #117

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

Conversation

sonicSMASH
Copy link

This enables signs from the street_signs mod to use this mod's poles as a pole to correctly attach to.

This enables signs from the street_signs mod to use this mod's poles as a pole to correctly attach to.
@@ -11,7 +11,7 @@ minetest.register_node(":streets:bigpole", {
drawtype = "nodebox",
tiles = { "streets_pole.png" },
sunlight_propagates = true,
groups = { cracky = 1, level = 2, bigpole = 1, not_blocking_trains = 1},
Copy link
Member

@SmallJoker SmallJoker Nov 12, 2023

Choose a reason for hiding this comment

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

What does not_blocking_trains do? Why can it be removed?

EDIT: Added in #109

@Emojigit What does this do?

Copy link
Author

Choose a reason for hiding this comment

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

I don't know what it's there for, I didn't add it for any reason.

Copy link
Member

Choose a reason for hiding this comment

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

This is here for advtrains. If a block is registered with this group, trains will not be blocked even if this block is in its path. This is useful as poles are not full blocks, hence not visually blocking trains but technically would.

Copy link
Author

Choose a reason for hiding this comment

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

Alright, it got added back.

streets_poles/init.lua Outdated Show resolved Hide resolved
mod.conf Outdated Show resolved Hide resolved
sonicSMASH and others added 3 commits December 7, 2023 20:41
I am untrained in version numbers, an explanation to why it goes from 2.0.1 to 2.1.0 would be nice
@SmallJoker
Copy link
Member

@sonicSMASH There a few nodes that still have the not_blocking_trains = 1 group removed. Is that intentional or did you forget to undo those changes?

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.

3 participants