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

Do not remove death message unless player is in a game #280

Merged
merged 1 commit into from
Feb 20, 2022
Merged

Do not remove death message unless player is in a game #280

merged 1 commit into from
Feb 20, 2022

Conversation

nicegamer7
Copy link
Contributor

This PR just changes the PlayerDeathEvent listener so that it doesn't remove the death message unless the player is actually in a game.

@andrei1058
Copy link
Owner

remove the message if the player dies in lobby world as well

@andrei1058 andrei1058 added the requires-changes Pull requests which need to be updated based on review comments and then reviewed again. label Jan 29, 2022
@nicegamer7
Copy link
Contributor Author

Sure, I'll change that.

@nicegamer7
Copy link
Contributor Author

Sorry for the delay.

Should I also add a check for MULTIARENA for the lobby? Or is this fine?

@andrei1058
Copy link
Owner

(BedWars.getLobbyWorld().equals(e.getEntity().getWorld().getName()) and MULTIARENA) otherwise if you switch to SHARED it will keep disabling kill message in lobby world since it is still in config.yml

@nicegamer7
Copy link
Contributor Author

Any blockers on this? Or is it good to go?

@andrei1058 andrei1058 merged commit 3b4a81d into andrei1058:master Feb 20, 2022
Cubixmeister pushed a commit to kwadratowa/BedWars1058 that referenced this pull request Apr 30, 2022
dhiepp pushed a commit to dhiepp/BedWars1058 that referenced this pull request Jul 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
requires-changes Pull requests which need to be updated based on review comments and then reviewed again.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants