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

Is there any use of messagecooldown and teleportcooldown in config? #2424

Open
benwoo1110 opened this issue Oct 5, 2020 · 5 comments · May be fixed by #2824
Open

Is there any use of messagecooldown and teleportcooldown in config? #2424

benwoo1110 opened this issue Oct 5, 2020 · 5 comments · May be fixed by #2824
Labels
Bug: Unconfirmed Unsure if the issue/bug is valid.

Comments

@benwoo1110
Copy link
Member

Description
After someone at discord ask about it, I look through the code brief relating to messagecooldown and teleportcooldown config options, and it doesn't seem to do anything. Am I missing something or is it a feature removed?

@benwoo1110 benwoo1110 added Bug: Confirmed Issue/problem with the software. Bug: Unconfirmed Unsure if the issue/bug is valid. labels Oct 5, 2020
@nicegamer7
Copy link
Member

nicegamer7 commented Oct 5, 2020

Teleport cooldown is used here and message cooldown is used here.

@benwoo1110
Copy link
Member Author

benwoo1110 commented Oct 5, 2020

yea i saw that, but the getTeleportable() method isn't used anywhere, and it is supposed to affect "How fast are people allowed to use /MVTP" according to the wiki https://github.com/Multiverse/Multiverse-Core/wiki/config.yml But when I change the value to like 100000 it doesn't affect my ability to do mvtp as well

@nicegamer7
Copy link
Member

I see, you're right...

@nicegamer7
Copy link
Member

Does the message cooldown work?

@benwoo1110
Copy link
Member Author

From my testing, message cooldown doesn't have any effect as well.

@benwoo1110 benwoo1110 removed the Bug: Confirmed Issue/problem with the software. label Oct 12, 2020
@benwoo1110 benwoo1110 linked a pull request Feb 10, 2023 that will close this issue
75 tasks
@zax71 zax71 linked a pull request Jan 12, 2025 that will close this issue
75 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Unconfirmed Unsure if the issue/bug is valid.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants