Skip to content

fix: wait for the engine v2 migration to save before reloading - #2438

Merged
kpal81xd merged 1 commit into
mainfrom
fix/engine-v2-migration-reload
Oct 26, 2026
Merged

kpal81xd merged 1 commit into
mainfrom
fix/engine-v2-migration-reload

Conversation

@kpal81xd

Copy link
Copy Markdown
Contributor

What's Changed

  • Opening an engine v1 project with write access sets engineV2 and reloads. The reload ran before the server had the change, so it was lost and every load migrated and reloaded again (3–4 reloads before settling). The Editor now waits until the project settings and the current scene have no pending changes, then reloads once.
  • Adds whenNothingPending to the settings observer.

Smoke test:

  1. Open a project whose project settings have engineV2 unset or false. Expected: the Editor reloads once, then stays open.
  2. Reload the page. Expected: no further migration reload; Project Settings shows engine v2.
  3. Open the same project as a read-only user. Expected: no reload.

Checks

@vercel

vercel Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
editor Ready Ready Preview Oct 26, 2026 10:00am UTC

Request Review

@kpal81xd kpal81xd self-assigned this Oct 26, 2026
@kpal81xd kpal81xd added bug Something isn't working release: next patch Ticket marked for next patch release labels Oct 26, 2026
@kpal81xd
kpal81xd marked this pull request as ready for review October 26, 2026 10:21
@kpal81xd
kpal81xd merged commit 64fb9a8 into main Oct 26, 2026
10 of 12 checks passed
@kpal81xd
kpal81xd deleted the fix/engine-v2-migration-reload branch October 26, 2026 10:22
playcanvas-bot Bot pushed a commit that referenced this pull request Oct 26, 2026

This branch was successfully deployed

1 active deployment
Preview — d83dd4fe Deployed Oct 26, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working release: next patch Ticket marked for next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant