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

Fix spurious warning being raised about USDZ texture import and allow for user-defined sourceImages path #1279

Merged
merged 3 commits into from
Apr 2, 2021

Conversation

ysiewappl
Copy link
Contributor

@ysiewappl ysiewappl commented Mar 26, 2021

Hi:

This PR fixes #1273 and only shows the warning when the relevant importUSDZTextures flag is actually specified.

Update: I've also updated this PR to also fix #1272 as well since the code changes are minimal and they both relate to the same functionality.

Please review and raise any concerns.

@kxl-adsk kxl-adsk added the import-export Related to Import and/or Export label Mar 26, 2021
@ysiewappl ysiewappl changed the title Fix spurious warning being raised about USDZ texture import Fix spurious warning being raised about USDZ texture import and allow for user-defined sourceImages path Mar 27, 2021
@JGamache-autodesk JGamache-autodesk added the ready-for-merge Development process is finished, PR is ready for merge label Mar 29, 2021
@kxl-adsk kxl-adsk merged commit 699957e into Autodesk:dev Apr 2, 2021
@ysiewappl ysiewappl deleted the spurious-warning branch April 17, 2021 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
import-export Related to Import and/or Export ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
4 participants