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

Add path to missing import texture metadata to error message #80107

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

Spycemyster
Copy link
Contributor

Before, the error message would just warn that a texture needed to be reimported but would not list which texture. This change adds the file path to the error output for the Missing required editor-specific import metadata for a texture; please, reimport message and updates the sentence structure.

Now changed to Missing required editor-specific import metadata for a texture (please reimport it using the 'Import' tab): FILE_PATH

fixes: #79550

@Spycemyster Spycemyster requested a review from a team as a code owner August 1, 2023 08:32
@AThousandShips AThousandShips requested review from fire and clayjohn August 1, 2023 08:33
@AThousandShips AThousandShips added enhancement topic:editor cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release labels Aug 1, 2023
@AThousandShips AThousandShips added this to the 4.2 milestone Aug 1, 2023
@Spycemyster
Copy link
Contributor Author

Apologies, accidentally closed the last PR for this change. Recreated here on a separate branch.

Here's the original PR: #79744

@AThousandShips AThousandShips changed the title Added path to output of missing import texture metadata Add path to the output of missing import texture metadata Aug 1, 2023
@AThousandShips AThousandShips changed the title Add path to the output of missing import texture metadata Add path to missing import texture metadata to error message Aug 1, 2023
@YuriSizov YuriSizov merged commit 2bd1d8a into godotengine:master Aug 1, 2023
@YuriSizov
Copy link
Contributor

Thanks! And congrats on your first merged Godot PR!

For the record, if you accidentally close a PR, you can always reopen it and push any changes you want to the same branch.

@Spycemyster
Copy link
Contributor Author

Thanks! And congrats on your first merged Godot PR!

Thanks and hopefully more to come soon =D

For the record, if you accidentally close a PR, you can always reopen it and push any changes you want to the same branch.

Will keep this in mind!

@Spycemyster Spycemyster deleted the reimport-path-message branch August 1, 2023 19:18
@YuriSizov
Copy link
Contributor

Cherry-picked for 4.1.2.

@YuriSizov YuriSizov removed the cherrypick:4.1 Considered for cherry-picking into a future 4.1.x release label Sep 21, 2023
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.

Import error message doesn't cite the involved resources
3 participants