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

Ftrack: update asset names for multiple reviewable items#6077

Merged
jakubjezek001 merged 2 commits intodevelopfrom
bugfix/ftrack-multiple-reviewables-missing
Dec 20, 2023
Merged

Ftrack: update asset names for multiple reviewable items#6077
jakubjezek001 merged 2 commits intodevelopfrom
bugfix/ftrack-multiple-reviewables-missing

Conversation

@jakubjezek001
Copy link
Copy Markdown
Member

Changelog Description

Multiple reviewable assetVersion components with better grouping to asset version name.

Additional info

  • Add a condition to also check if multiple_reviewable is True before updating the asset name.

Testing notes:

  1. very difficult to test but basically you need multiple reviewable representations marked as ftrackreview.

…iple reviewable items

- Add a condition to also check if multiple_reviewable is True before updating the asset name.
@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files module: Ftrack type: bug Something isn't working labels Dec 20, 2023
Copy link
Copy Markdown
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.

LGTM

@jakubjezek001 jakubjezek001 merged commit 9ea9096 into develop Dec 20, 2023
@ynbot ynbot added this to the next-patch milestone Dec 20, 2023
@mkolar mkolar deleted the bugfix/ftrack-multiple-reviewables-missing branch January 25, 2024 22:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

module: Ftrack size/XS Denotes a PR changes 0-99 lines, ignoring general files target: OpenPype type: bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants