Skip to content

Review: use new reviewables endpoint for review creation - #2186

Draft
Innders wants to merge 5 commits into
developfrom
2175-Review-Use-new-reviewables-endpoint-for-review-creation
Draft

Innders wants to merge 5 commits into
developfrom
2175-Review-Use-new-reviewables-endpoint-for-review-creation

Conversation

@Innders

@Innders Innders commented Jul 20, 2026

Copy link
Copy Markdown
Member

Description of changes

"Create new session" and "Add to session" now use the new reviewables endpoint instead of relying on the review addon actions.

Before the changes, we would use different actions provided by the review session to get the latest reviewables of a folder/task. Now it uses a dedicated endpoint ynput/ayon-backend#994 (review)

image

Validation

  • Always max 1 version per selected entity.
  • 3 folders = 3 latest versions, from each folder.
  • If the latest version does not have a reviewable then it is skipped. 5 tasks = 3 versions (with reviewables).
  • Needs to work for folders, tasks, versions.

Testing

  1. Go to overview page
  2. Select some folders that have the play icon in the thumbnail.
  3. Right click review / create new session
  4. The latest version from each folder should be in the new list. If the version does NOT have a reviewable, it is skipped.
  5. Repeat for Add to session
  6. Repeat for each entity type folder, task, version`.
  7. Test on each page overview, task progress, products, lists, review

@Innders Innders self-assigned this Jul 20, 2026
@Innders Innders added the type: maintenance Changes to the code that don't affect product functionality (Technical debt, refactors etc.)) label Jul 20, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: maintenance Changes to the code that don't affect product functionality (Technical debt, refactors etc.))

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant