Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename FBX2glTF binary path setting back to 4.2 name #92571

Merged

Conversation

akien-mga
Copy link
Member

@akien-mga akien-mga commented May 30, 2024

This preserves compatibility when upgrading Godot 4.2 projects which relied on that path being configured in the editor settings.

The old name also makes sense for this one, it's fine for fbx2gltf_path to be under a generic fbx category which could have more settings also impacting ufbx. I was the one doing this rename as part of cleaning up #81746, my bad.

I didn't change the project setting filesystem/import/fbx2gltf/enabled back to fbx/enabled because this one would really be confusing. But it's also enabled by default in 4.2, so this won't change anything for users who do use fbx2gltf. For those who turned it off, they'll experience either way that their fbx files are now imported by ufbx in 4.3.

This preserves compatibility when upgrading Godot 4.2 projects which relied on that
path being configured in the editor settings.

The old name also makes sense for this one, it's fine for fbx2gltf_path to be under
a generic fbx category which could have more settings also impacting ufbx.
@akien-mga akien-mga added this to the 4.3 milestone May 30, 2024
@akien-mga akien-mga requested review from a team as code owners May 30, 2024 21:42
@akien-mga akien-mga merged commit a4f2ea9 into godotengine:master May 30, 2024
15 checks passed
@akien-mga akien-mga deleted the fbx2gltf-rename-path-setting-back branch May 30, 2024 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant