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

Mitigate chests from spawning in air #19

Merged
merged 1 commit into from Jun 14, 2020

Conversation

XDleader555
Copy link
Contributor

Prevent the chest from spawning on certain blocks (Like grass paths)
if it's within our radius. Modify method for easy changing in the future.

Prevent the chest from spawning on certain blocks (Like grass paths)
if it's within our radius. Modify method for easy changing in the future.
@mfnalex mfnalex merged commit 79062f2 into mfnalex:master Jun 14, 2020
@mfnalex
Copy link
Owner

mfnalex commented Jun 14, 2020

Thanks for all the PRs :) I will add a few other stings and then finally release a new version. Would you like to be credited as contributor at SpigotMC? :)

@XDleader555
Copy link
Contributor Author

Thanks for all the PRs :) I will add a few other stings and then finally release a new version. Would you like to be credited as contributor at SpigotMC? :)

Sure, that'd be cool. I have more features which players on my server wanted which may make it over as PRs.

@mfnalex
Copy link
Owner

mfnalex commented Jun 15, 2020

Okay, the next update will be released today probably :)

What other features did you add? Unfortunately I changed some stuff and moved files so I don‘t know if it‘s easily possible to merge commits from your fork anymore :/

@mfnalex
Copy link
Owner

mfnalex commented Jun 15, 2020

Your Spigot username is the same as here?

@XDleader555
Copy link
Contributor Author

Same username. Only things I haven't submitted is instead of teleporting to the chest, the chest can be teleported to the player and remove curse of vanishing items. I can handle merging with your code.

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