Skip to content

/Sit Command (For Tank) - #35

Open
Fishywishi wants to merge 17 commits into
Warzone:masterfrom
Fishywishi:sit
Open

/Sit Command (For Tank)#35
Fishywishi wants to merge 17 commits into
Warzone:masterfrom
Fishywishi:sit

Conversation

@Fishywishi

Copy link
Copy Markdown

Allow players to use /sit command to sit down on the ground. Players in matches can use command but are unseated and take knockback as normal when hit

Comment thread src/main/kotlin/network/warzone/mars/Mars.kt Outdated
Comment thread src/main/kotlin/network/warzone/mars/player/listeners/tasks/ChairRotationTask.kt Outdated
Comment thread src/main/kotlin/network/warzone/mars/player/listeners/tasks/ChairRotationTask.kt Outdated
Comment thread src/main/kotlin/network/warzone/mars/player/controllers/SitController.kt Outdated
Comment thread src/main/kotlin/network/warzone/mars/Mars.kt Outdated
Comment thread src/main/kotlin/network/warzone/mars/player/controllers/SitController.kt Outdated

@jorgeberrex jorgeberrex left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I like this idea but I feel like this is outside of the scope for Mars. As this feature does not rely on any kind of direct integration with Mars, I would propose moving this functionality to an entirely new plugin where some other fun features could be added. This way we can keep Mars strictly for server management and version-independent.

import network.warzone.mars.player.controllers.SitController
import org.bukkit.Bukkit
import org.bukkit.Location
import org.bukkit.craftbukkit.v1_8_R3.entity.CraftArmorStand

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This PR would break compatibility for modern server versions since you are directly using craftbukkit imports in some files.
Currently Mars does not have any utility functions for handling NMS/OBC imports so that would need to be implemented as well

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.

5 participants