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

Avoid error spam on first opening of a not yet imported project #77276

Merged

Conversation

RandomShaper
Copy link
Member

@akien-mga, I have the impression that you will especially appreciate this one.

Besides, this affects every dependency collection operation started by the EditorFileSystem, so maybe there are some cases where the error shouldn't be silenced. In any case, I can't think of a situation where they would be missed much.

@akien-mga
Copy link
Member

Omg I've been waiting for this since 3.0 alpha days... You rock! 🎉

@akien-mga
Copy link
Member

I gave it a try, and it seems to work great!

I did get spam of this error:

ERROR: Attempting to parent and popup a dialog that already has a parent.
   at: _try_parent_dialog (scene/main/window.cpp:1670)

but I don't think it's related to this PR.

@akien-mga akien-mga added cherrypick:4.0 cherrypick:3.x Considered for cherry-picking into a future 3.x release labels May 22, 2023
@akien-mga akien-mga merged commit 6a3095a into godotengine:master May 22, 2023
@akien-mga
Copy link
Member

Thanks! 🎉 🥳

@RandomShaper RandomShaper deleted the avoid_first_import_error_spam branch May 22, 2023 15:13
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.0.4. This is a pretty important usability fix to miss out on, but we should also be ready to revert the pick in case this compromises existing projects.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:editor usability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants