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

Load: Containerize project_name on load to allow managing containers from different projects #176

Merged

Conversation

BigRoy
Copy link
Contributor

@BigRoy BigRoy commented Nov 12, 2024

Changelog Description

Maya imprint and update with project_name on container object.

Additional review information

Should be tested together with PR ynput/ayon-core#1005

Note that this would only set project name on update for ReferenceLoader plug-ins and not the other, so that would still be TODO. However, that'd only be relevant when opening a scene from another project OR when being able to 'switch asset' to another project.

Testing notes:

  1. Use with PR Load and manage products from a library project ayon-core#1005
  2. Load containers from library project into current project.
  3. They should be manageable via the scene inventory.

OR alternatively:

  1. Load containers from current project.
  2. Save workfile.
  3. Launch maya from another project - secretly open the file from the first project.
  4. It should list and manage the containers just fine in the scene inventory because it now knows about its project.

- Note that this would only set project name on update for `ReferenceLoader` plug-ins and not the other, so that would still be TODO
@BigRoy BigRoy added the type: enhancement Improvement of existing functionality or minor addition label Nov 12, 2024
@BigRoy BigRoy self-assigned this Nov 12, 2024
Copy link
Member

@iLLiCiTiT iLLiCiTiT left a comment

Choose a reason for hiding this comment

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

I'd say this needs validation of @antirotor . But if we do want to support per project loading, then, yes?

@moonyuet moonyuet merged commit 2381a85 into ynput:develop Dec 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants