Skip to content

Comments

fix: movePlayerTo() with Duration#7089

Merged
pravusjif merged 2 commits intodevfrom
fix/move-player-to-duration
Feb 18, 2026
Merged

fix: movePlayerTo() with Duration#7089
pravusjif merged 2 commits intodevfrom
fix/move-player-to-duration

Conversation

@pravusjif
Copy link
Member

@pravusjif pravusjif commented Feb 15, 2026

WHY

Currently the usage of SDK's global function movePlayerTo() with its recently implemented duration parameter is completely broken: when used, after the inteprolation of the avatar finishes, the avatar gets repositioned to the original position, snapping back to the place it was before calling the movePlayerTo()...

Easily reproductible in the exitent feature dedicated test scene: https://github.com/decentraland/sdk7-test-scenes/tree/main/scenes/11%2C0-move-player-to-duration

Issue: #6916

WHAT

  • Fix movePlayerTo() (with duration) snapping back to original position before the function call, by adding a missing platform value reset.
  • Updated test coverage

TEST INSTRUCTIONS

  1. Clone the sdk7-test-scenes repo (or make sure you download its latest changes) and enter this test scene
  2. In the scene root folder open a terminal and run npm i and then npm run start -- --explorer-alpha
  3. Close the Explorer that auto-opened. Leave the scene running in the console/terminal.
  4. Download the build from this PR and open it connected to the local scene (keep in mind its position is 11,0): https://github.com/decentraland/unity-explorer/wiki/How-to-connect-to-a-local-scene-(Unity-Editor---Custom-Build---Latest-Released-Build)#connecting-a-custom-build-to-the-scene
  5. Once you enter the scene confirm that:
  • When you click on the little cube in the floor, your avatar automatically moves to it
  • The cube gets randomly re-positioned when the avatar finishes moving
  • you DON'T get automatically repositioned to the original position before moving to the cube

@pravusjif pravusjif self-assigned this Feb 15, 2026
@pravusjif pravusjif added the bug Something isn't working label Feb 15, 2026
@pravusjif pravusjif added 1-high Severely degrades primary features. Workaround may exist but user experience is very poor. sdk labels Feb 15, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 15, 2026

@pravusjif pravusjif moved this from Todo to With QA / Awaiting Review in Explorer Alpha Feb 15, 2026
@pravusjif pravusjif marked this pull request as ready for review February 15, 2026 20:12
@pravusjif pravusjif requested review from a team as code owners February 15, 2026 20:12
@decentraland decentraland deleted a comment from github-actions bot Feb 15, 2026
@decentraland decentraland deleted a comment from github-actions bot Feb 15, 2026
@pravusjif pravusjif removed the request for review from mikhail-dcl February 15, 2026 20:12
@pravusjif pravusjif requested a review from DafGreco February 17, 2026 02:49
@pravusjif pravusjif enabled auto-merge (squash) February 17, 2026 15:21
Copy link

@DafGreco DafGreco left a comment

Choose a reason for hiding this comment

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

✔️ PR reviewed and approved by QA on both platforms following instructions playing both happy and un-happy path

Regressions for this ticket had been performed in order to verify that the normal flow is working as expected:

  • [ ✔️] Backpack and wearables in world
  • [ ✔️] Emotes in world and in backpack
  • [✔️ ] Teleport with map/coordinates/Jump In
  • [✔️ ] Chat and multiplayer
  • [ ✔️] Profile card
  • [✔️ ] Camera
  • [✔️ ] Skybox
  • [✔️ ] Settings

Evidence:
link to evidence of macos

20260218-1614-25.4183811.mp4

Implementation and cases tested :

  1. Jumping and clicking the boxes
  2. Multiple Jumping and clicking
  3. Getting outside the scene and returning

No new issues were found during this checkup

@github-project-automation github-project-automation bot moved this from With QA / Awaiting Review to In Progress in Explorer Alpha Feb 18, 2026
@pravusjif pravusjif merged commit 83e2b2d into dev Feb 18, 2026
9 of 11 checks passed
@pravusjif pravusjif deleted the fix/move-player-to-duration branch February 18, 2026 16:28
@github-project-automation github-project-automation bot moved this from In Progress to Done in Explorer Alpha Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-high Severely degrades primary features. Workaround may exist but user experience is very poor. bug Something isn't working sdk

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants