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

Photoshop: fix creation of .mov#6064

Merged
kalisp merged 1 commit intodevelopfrom
bugfix/OP-7606_ExtractReview-fails-when-creating-mov-file
Dec 18, 2023
Merged

Photoshop: fix creation of .mov#6064
kalisp merged 1 commit intodevelopfrom
bugfix/OP-7606_ExtractReview-fails-when-creating-mov-file

Conversation

@kalisp
Copy link
Copy Markdown
Member

@kalisp kalisp commented Dec 18, 2023

Changelog Description

Generation of .mov file with 1 frame per published layer was failing.

Testing notes:

  1. set ayon+settings://photoshop/publish/ExtractReview?project=demo_Commercial Make an image sequence instead of flatten image

image

  1. publish workfile with multiple image instances
  2. it shouldn't fail

@kalisp kalisp requested a review from 64qam December 18, 2023 13:10
@ynbot
Copy link
Copy Markdown
Contributor

ynbot commented Dec 18, 2023

@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files host: Photoshop type: bug Something isn't working labels Dec 18, 2023
@kalisp kalisp assigned LiborBatek and unassigned 64qam Dec 18, 2023
@LiborBatek LiborBatek self-requested a review December 18, 2023 14:07
Copy link
Copy Markdown
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.

All working as expected...for 3 image products,

it does produces mov file with 3 frames each representing image instance.

image

@kalisp kalisp merged commit 0252c9e into develop Dec 18, 2023
@kalisp kalisp deleted the bugfix/OP-7606_ExtractReview-fails-when-creating-mov-file branch December 18, 2023 15:02
@ynbot ynbot added this to the next-patch milestone Dec 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

host: Photoshop size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants