Skip to content

Switch scene FUnityManager to new project on creation#356

Merged
oco777 merged 1 commit intomainfrom
codex/add-project-switch-for-funitymanager-on-new-project
Nov 22, 2025
Merged

Switch scene FUnityManager to new project on creation#356
oco777 merged 1 commit intomainfrom
codex/add-project-switch-for-funitymanager-on-new-project

Conversation

@oco777
Copy link
Copy Markdown
Owner

@oco777 oco777 commented Nov 22, 2025

Summary

  • update project creation window to set the active FUnityManager project to the newly created project data
  • record undo and mark manager dirty so the change can be reverted and saved

Testing

  • not run

Codex Task

@oco777 oco777 merged commit 8d44477 into main Nov 22, 2025
3 checks passed
@oco777 oco777 deleted the codex/add-project-switch-for-funitymanager-on-new-project branch November 22, 2025 10:58
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ 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".

Comment on lines +156 to +157
Undo.RecordObject(manager, "Switch FUnity Project");
manager.Project = projectData;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P0 Badge Assignment uses undefined FUnityManager.Project property

The new undo block assigns manager.Project = projectData, but FUnityManager exposes only the ProjectData getter and the editor-only Editor_SetProjectData(...) helper in Runtime/Core/FUnityManager.cs and has no writable Project member. This code will not compile in the editor assembly, blocking the entire project build until the assignment is rewritten to use the available API.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant