Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Add shortcut to open template for current context #6181

Closed
wants to merge 18 commits into from

Conversation

tokejepsen
Copy link
Member

Changelog Description

Replacement for #4953 with resolved comments.

Testing notes:

  1. Add a template for your context in the settings
  2. Click on open template

    237122104-d51be424-19ef-47d7-81b7-7e0d6a69b595

@ynbot ynbot added type: enhancement Enhancements to existing functionality host: Maya host: Nuke size/S Denotes a PR changes 100-499 lines, ignoring general files labels Jan 30, 2024
@tokejepsen
Copy link
Member Author

@ClementHector could you have a look as well?

@LiborBatek
Copy link
Member

LiborBatek commented Feb 5, 2024

Maybe offtopic but how about to add rollout menu with premade template files as well a list when triggering

Build Work File from Template > Template_Lighting_Outdoors.ma
.......................................................................Template_Lighting_Interior.ma etc

It would be pretty useful to be able to have multiple templates available (listing any maya scene at that location for current context)

Just an idea maybe for other PR?

@tokejepsen
Copy link
Member Author

Just an idea maybe for other PR?

@LiborBatek could you make an issue for this?
Wont be part of this PR.

@BigRoy
Copy link
Collaborator

BigRoy commented Feb 5, 2024

Maybe offtopic but how about to add rollout menu with premade template files as well a list when triggering

Build Work File from Template > Template_Lighting_Outdoors.ma .......................................................................Template_Lighting_Interior.ma etc

It would be pretty useful to be able to have multiple templates available (listing any maya scene at that location for current context)

Just an idea maybe for other PR?

Honestly at that point we're maybe better of implementing ynput/ayon-core#742
That way it would at least be as easy as browsing via the Loader or alike.

@LiborBatek
Copy link
Member

@BigRoy totally yes! ...I have not been aware of that one. thx

@tokejepsen
Copy link
Member Author

@LiborBatek did you test this? Could you approve if its tested?

Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

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

Did tests with 2 context Template files...for animation and lighting task types.

Also tested with hardcoded file path to the template_file and file paths with tokens

image

All working fine, after triggering Open Template File you get into the template and being able to adjust it and resave it. Tested then with building the work file with that updated template file...

All correctly built.
image

@tokejepsen tokejepsen assigned antirotor and unassigned LiborBatek Feb 9, 2024
@mkolar
Copy link
Member

mkolar commented Feb 9, 2024

Because we're splitting OpenPype into ayon-core and individual host addons, this PR would have to be re-created to target one of those.

We're closing it down, but we'll he happy for a new PR to ynput/ayon-core or the host addon repository once it's up.

@mkolar mkolar closed this Feb 9, 2024
@ynbot ynbot added this to the next-patch milestone Feb 9, 2024
@jakubjezek001 jakubjezek001 removed this from the next-patch milestone Feb 15, 2024
@tokejepsen tokejepsen deleted the enhancement/shortcut_to_open_template branch February 20, 2024 11:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Maya host: Nuke port to AYON size/S Denotes a PR changes 100-499 lines, ignoring general files target: AYON target: OpenPype type: enhancement Enhancements to existing functionality
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

9 participants