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

Fireball Enhancement #804

Closed
wants to merge 5 commits into from
Closed

Fireball Enhancement #804

wants to merge 5 commits into from

Conversation

Lsy-291
Copy link
Contributor

@Lsy-291 Lsy-291 commented Jun 11, 2023

Fireball can now destroy blocks and supports the following functions

  1. Custom destruction radius
  2. Whitelist
  3. Flip the whitelist
  4. Destroy existing blocks in the map
  5. The damage range of the block is a sphere
  6. Custom Area Protection
  7. Explosion particle effect

@Lsy-291 Lsy-291 temporarily deployed to development June 12, 2023 22:27 — with GitHub Actions Inactive
@andrei1058
Copy link
Owner

Isn't this related to #469 ? We should implement the protection logic for fireballs there.

@Lsy-291
Copy link
Contributor Author

Lsy-291 commented Jul 1, 2023

这与#469无关吗?我们应该在那里实现火球的保护逻辑。

Now it has not achieved support for explosion-proof glass

@Lsy-291
Copy link
Contributor Author

Lsy-291 commented Jul 1, 2023

这与#469无关吗?我们应该在那里实现火球的保护逻辑。

Do you mean that this PR lacks support for explosion-proof glass?

@andrei1058
Copy link
Owner

andrei1058 commented Jul 1, 2023

The other pull request will also protect cumstom areas such as shop etc. Fireballs are explosions, so in the end I think I will implement your logic in the other pull request to make it cleaner.

@Lsy-291
Copy link
Contributor Author

Lsy-291 commented Jul 1, 2023

The other pull request will also protect cumstom areas such as shop etc. Fireballs are explosions, so in the end I think I will implement your logic in the other pull request to make it cleaner.

So it won't be merged?

@andrei1058 andrei1058 added the on-hold No time or waiting for something else. label Jul 16, 2023
@Lsy-291 Lsy-291 closed this by deleting the head repository Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on-hold No time or waiting for something else.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants