Skip to content

Conversation

@TheDrawingCoder-Gamer
Copy link
Contributor

adds config option for whether items fly out of your inventory upon using greater teleport

closes #502

Copy link
Member

@object-Object object-Object left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added one suggestion; otherwise it looks good. Tested on Fabric and Forge, seems to work fine.

@object-Object
Copy link
Member

oh, it seems like there are some conflicts now

@TheDrawingCoder-Gamer
Copy link
Contributor Author

do I rebase and close and reopen this PR?

@object-Object
Copy link
Member

a merge commit would have been better, but I guess that works

@object-Object
Copy link
Member

you shouldn't need to close and reopen the PR.

@TheDrawingCoder-Gamer
Copy link
Contributor Author

Oops, broke the commit I just applied in github, LOL

@object-Object
Copy link
Member

I haven't tested ingame after the rebase, but I'm going to lazily merge it in anyway because I don't see why it would have broken (fingers crossed). Thanks for the PR!

@object-Object object-Object merged commit ef2cd28 into FallingColors:main Feb 4, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

[Feature request] Make greater teleport dropping items into a gamerule and/or config option

2 participants