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

Make Reinforced Cobblestone harder to break for enemy team #1250

Merged
merged 5 commits into from
Dec 5, 2023

Conversation

a-blob
Copy link
Contributor

@a-blob a-blob commented Nov 27, 2023

Make it so that Reinforced Cobblestone is easier to break for the team that placed it but harder to break for other teams.

@LoneWolfHT
Copy link
Member

This works? Feels a little hacky.
It might be better to make all reinforced cobble the hardened version, but have enemies dig it twice by replacing it with a second version of itself one dug by an enemy that has a cracked texture applied. Node dig predictions should be considered with that approach

@a-blob
Copy link
Contributor Author

a-blob commented Nov 28, 2023

A bit hacky, but works very well. I'd rather stick to this approach unless there's some unforeseen problem that might cause issues. Currently it is not complete though.

@a-blob a-blob marked this pull request as ready for review November 28, 2023 02:00
@LoneWolfHT
Copy link
Member

How does this work with internet lag? You might be able to simulate it with minetest.after()

@a-blob
Copy link
Contributor Author

a-blob commented Nov 29, 2023

The typical internet lag doesn't seem to affect it much. Bad internet lag (1+ seconds) makes it seem like it is dug "twice by replacing it with a second version of itself once dug by an enemy".

@LoneWolfHT LoneWolfHT merged commit b66c0d6 into MT-CTF:master Dec 5, 2023
1 check passed
@LoneWolfHT
Copy link
Member

I suspect something is going to turn up requiring a change, but I'll go with it as-is for now

@a-blob a-blob deleted the reinforce-cobble branch January 29, 2024 02:34
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

2 participants