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

Refactors lava into being a component with different types for toxic water and plasma rivers #22017

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

Conversation

Moltijoe
Copy link
Contributor

@Moltijoe Moltijoe commented May 6, 2024

it was initially just copying code from lava over to toxic water
but i decided it would be better if refactored into a component
so now it is

Why is this good for the game?

code quality + consistency between similar features
also, it makes it easier to create new types of terrain that do stuff to people standing on it
(could make healing terrain that heals instead of doing damage)

Testing

image
image

🆑
tweak: Refactors lava into being a component with different types for toxic water and plasma rivers
bugfix: can now rcd toxic water
bugfix: boats and grilles now protect from toxic water
experimental: please tell me DIRECTLY if there's anything wonky about lava/plasma/toxic water
/:cl:

@Yogbot-13 Yogbot-13 added Fix This fixes an issue. Please link issues in fix PRs Tweak This PR contains changes to the game. labels May 6, 2024
@Moltijoe Moltijoe added the Awaiting - Action - Testing TEST YOUR STUFF label May 6, 2024
@Moltijoe Moltijoe marked this pull request as draft May 26, 2024 00:36
@Moltijoe Moltijoe changed the title Modifies toxic water on jungleland to function more like lava Refactors lava into being a component with different types for toxic water and plasma rivers May 26, 2024
@Moltijoe Moltijoe removed the Awaiting - Action - Testing TEST YOUR STUFF label May 26, 2024
@Moltijoe Moltijoe marked this pull request as ready for review May 26, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix This fixes an issue. Please link issues in fix PRs Tweak This PR contains changes to the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants