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-106503 load all prims in hierarchy view. #1140

Merged
merged 1 commit into from
Feb 2, 2021

Conversation

pierrebai-adsk
Copy link
Collaborator

No description provided.

@kxl-adsk kxl-adsk requested review from seando-adsk and removed request for kxl-adsk February 1, 2021 23:29
@kxl-adsk kxl-adsk added the import-export Related to Import and/or Export label Feb 1, 2021
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.

I'm pretty sure that the LoadNone was intentional as initially we didn't want theses prims. But its fine to change it now if the specs changed.

@pierrebai-adsk
Copy link
Collaborator Author

IDK if the specs, changed, just that a bug was filed about this issue... if the LoadNone was intentional, the other option would be to close the bug as Äs Designed" instead. I don't think I should be the one to decide on design...

@seando-adsk
Copy link
Collaborator

Sorry I wasn't implying that you needed to make any decision. I was just pointing out that design team might have changed the spec. It was a long time ago I wrote this, so I don't recall. The change is fine with me.

@pierrebai-adsk pierrebai-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Feb 2, 2021
@kxl-adsk kxl-adsk merged commit 5551f2c into dev Feb 2, 2021
@kxl-adsk kxl-adsk deleted the t_bailp/MAYA-106503/hierarchy-view-missing-prims branch February 2, 2021 20:04
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
Development

Successfully merging this pull request may close these issues.

4 participants