Delete Assets/newdesign/Resources/phase2a directory - #85
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2ad48d6165
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| @@ -1,114 +0,0 @@ | |||
| fileFormatVersion: 2 | |||
| guid: c1071e21fb9d4d5780423b4fa2082ffb | |||
There was a problem hiding this comment.
Keep phase2a assets until all consumers migrate
At this commit, runtime owners for Play, Home, Settings, Private Room, onboarding, Solo, and PvP still call Resources.Load with phase2a/... paths, so deleting this directory makes those loads return null and leaves required backgrounds, buttons, heroes, and fonts absent; the checked tools/test/panelplay-assets.test.mjs also fails with ENOENT for this background. Restore these resources or migrate every loader and integrity test to replacement assets in the same change.
AGENTS.md reference: AGENTS.md:L88-L94
Useful? React with 👍 / 👎.
No description provided.