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

MAYA-108237 - As a user, on the prim AE template I'd like to see app… #1333

Merged
merged 3 commits into from
Apr 21, 2021

Conversation

spinell-adsk
Copy link
Contributor

…lied schemas

Add Applied Schemas section

Copy link
Collaborator

@seando-adsk seando-adsk left a comment

Choose a reason for hiding this comment

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

A couple of minor typos and two code comments.

I would also suggest verifying that all the UsdSchemaRegistry methods you are using are available all the way back to USD v20.02 (our min supported version). I have ran into this before.

@kxl-adsk kxl-adsk added the do-not-merge-yet Development is not finished, PR not ready for merge label Apr 16, 2021
…ied schemas

Disable AE applied schemas section for USD version < 0.21.2 because of Usd.SchemaRegistry().GetPropertyNamespacePrefix()
@spinell-adsk spinell-adsk added ready-for-merge Development process is finished, PR is ready for merge and removed do-not-merge-yet Development is not finished, PR not ready for merge labels Apr 20, 2021
@kxl-adsk kxl-adsk merged commit 2ba8eab into dev Apr 21, 2021
@kxl-adsk kxl-adsk deleted the spinell/MAYA-108237/ae_add_applied_schemas_section branch April 21, 2021 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants