Skip to content

Publish as AYON reviewable#18

Merged
Sasbom merged 4 commits into
ynput:developfrom
tweak-wtf:patch/upload-webreview
May 14, 2026
Merged

Publish as AYON reviewable#18
Sasbom merged 4 commits into
ynput:developfrom
tweak-wtf:patch/upload-webreview

Conversation

@tweak-wtf
Copy link
Copy Markdown
Contributor

This PR adds webreview tag to image/review products in order to automatically upload them to AYON as a reviewable representation.

Since this addon exclusively supports AYON web-reviewable media (png, mp4, webm) i think it's safe to always add the tag.

@tweak-wtf tweak-wtf changed the title adds webreview tag to main representation Publish as AYON reviewable May 6, 2026
Copy link
Copy Markdown
Member

@BigRoy BigRoy left a comment

Choose a reason for hiding this comment

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

Usually instead of directly passing on to webreview it goes through Extract Review (and perhaps Extract Burnin). For that it should:

  • Have 'review' family applied to the instance
  • Must have set instance.data["review"] = True
  • Must have "review" in representation["tags"] for the representation in instance.data["representations"] (so webreview -> review)
  • There is the caveat that currently Extract Review is ALSO hosts filtered 🤦‍♂️ but that's being addressed here: ynput/ayon-core#1810

I think if we make it go through 'review' steps it behaves more like other DCC integrations, and also has a higher chance the web review upload is of an optimized web playing format for the web reviews.

@iLLiCiTiT or would you approach otherwise?

@BigRoy BigRoy added the type: enhancement New feature or request label May 13, 2026
@iLLiCiTiT
Copy link
Copy Markdown
Member

It should go through Extract Review because the conversion for AYON server is more and more mandatory.

@Sasbom
Copy link
Copy Markdown
Collaborator

Sasbom commented May 14, 2026

Thanks for clarifying @BigRoy @iLLiCiTiT,
I've updated the PR, we'll merge right now.

@Sasbom Sasbom merged commit 66bd3e8 into ynput:develop May 14, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants