镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Fix warp messages sending incorrectly with teleport delays - #3696

Merged
JRoy merged 4 commits into
EssentialsX:2.xfrom
pop4959:warp-message-fix
Jan 2, 2021
Merged

JRoy merged 4 commits into
EssentialsX:2.xfrom
pop4959:warp-message-fix

Conversation

@pop4959

@pop4959 pop4959 commented Oct 2, 2020

Copy link
Copy Markdown
Member

Closes #1044

This PR sends the warp message only when the future for the teleport completes successfully (does not fail anything such as a cooldown, which then prints a separate message).

@pop4959 pop4959 added the type: enhancement Features and feature requests. label Oct 2, 2020

@JRoy JRoy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I wonder who caused this regression 😳

@JRoy

JRoy commented Oct 3, 2020

Copy link
Copy Markdown
Member

This may possibly fix #3554

@Chew Chew added status: needs manual rebase Needs a manual local rebase due to base changes (regardless of what GH says) and removed hacktoberfest-accepted labels Nov 26, 2020
@pop4959 pop4959 removed the status: needs manual rebase Needs a manual local rebase due to base changes (regardless of what GH says) label Nov 26, 2020
@mdcfe mdcfe added this to the 2.19.0 milestone Nov 27, 2020
@JRoy JRoy changed the title Fix warp messages sending incorrectly when there is a teleport delay Fix warp messages sending incorrectly with teleport delays Jan 2, 2021
@JRoy
JRoy merged commit d788324 into EssentialsX:2.x Jan 2, 2021
@pop4959
pop4959 deleted the warp-message-fix branch January 2, 2021 03:28
ressidell pushed a commit to ressidell/Essentials that referenced this pull request Apr 27, 2025
…sX#3696)

Co-authored-by: Josh Roy <10731363+JRoy@users.noreply.github.com>
rewsdjad1 pushed a commit to rewsdjad1/EssentialsXPlus that referenced this pull request Aug 13, 2026
…sX#3696)

Co-authored-by: Josh Roy <10731363+JRoy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement Features and feature requests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Warp Messages

4 participants