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

Report stagingDir_is_custom to apply_staging_dir #1084

Merged
merged 1 commit into from
Jan 14, 2025

Conversation

robin-ynput
Copy link
Contributor

Changelog Description

Following #1069 and testing ynput/ayon-nuke#52, I realized apply_staging_dir was missing stagingDir_is_custom.

Testing notes:

The following ensure Nuke product can be published with custom staging directories.

  1. Enable custom staging directory with some profiles ayon+settings://core/tools/publish/custom_staging_dir_profiles (make it persistent so you can troubleshoot)
  2. Create a template matching the staging directory under ayon+anatomy://Dummy/templates/staging
    e.g. {root[work]}/{project[name]}/{hierarchy}/{folder[name]}/temp_render_only/{product[type]}/{product[name]}
  3. Open Nuke and create+publish some products
  4. Ensure that the staging directory is working is properly redirected to the relevant template for the associated products/task

@ynbot ynbot added size/XS type: bug Something isn't working labels Jan 14, 2025
@robin-ynput robin-ynput requested a review from kalisp January 14, 2025 13:22
@robin-ynput robin-ynput self-assigned this Jan 14, 2025
@robin-ynput robin-ynput requested a review from iLLiCiTiT January 14, 2025 13:22
@iLLiCiTiT iLLiCiTiT changed the title Report stagingDir_is_custom to apply_staging_dir. Report stagingDir_is_custom to apply_staging_dir Jan 14, 2025
@iLLiCiTiT iLLiCiTiT merged commit 9b899eb into develop Jan 14, 2025
14 of 15 checks passed
@iLLiCiTiT iLLiCiTiT deleted the bugfix/report_stagingDir_is_custom branch January 14, 2025 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants