Skip to content

update rejoin - #453

Open
sleaze5 wants to merge 2 commits into
EdgeIY:masterfrom
sleaze5:patch-5
Open

update rejoin#453
sleaze5 wants to merge 2 commits into
EdgeIY:masterfrom
sleaze5:patch-5

Conversation

@sleaze5

@sleaze5 sleaze5 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Updated the rejoin command to include repositioning functionality.

Updated the rejoin command to include repositioning functionality.
Copilot AI review requested due to automatic review settings July 30, 2026 20:27

Copilot AI 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.

Pull request overview

This PR updates the rejoin command to optionally persist and restore the player’s position across the teleport by sending teleport data and running a queued respawn handler after rejoin.

Changes:

  • Extends rejoin / rj to accept an optional reposition flag and attempts to carry the current character pivot through TeleportService teleport data.
  • Adds a queued-on-teleport handler intended to re-pivot the character shortly after loading.
  • Updates the command list entry to reflect the new argument.
Comments suppressed due to low confidence (1)

source:6956

  • The reposition flag currently stores speaker.Character:GetPivot() directly into teleportData. If teleportData needs to be serializable, this should be encoded into primitives (e.g., CFrame components) to match the respawn handler's decode logic.
	local data = nil
	if parseBoolean(args[1], false) == true then
		if queueteleport then
			if speaker.Character then data = speaker.Character:GetPivot() end
			queueteleport(teleportRespawnHandler)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source
Comment thread source
@iiDk-the-inactual

Copy link
Copy Markdown

big fan of the teleport data set

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.

3 participants