Skip to content

Delete Versions: Allow to delete selected versions - #2034

Open
tadeas-hejnic wants to merge 9 commits into
developfrom
1804-yn-0686-loader-delete-versions
Open

Delete Versions: Allow to delete selected versions#2034
tadeas-hejnic wants to merge 9 commits into
developfrom
1804-yn-0686-loader-delete-versions

Conversation

@tadeas-hejnic

@tadeas-hejnic tadeas-hejnic commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Changelog Description

Adding Delete Selected Versions action in browser.

Additional info

image

Testing notes:

  1. select versions
  2. remove them
  3. check if everything went correctly

@tadeas-hejnic tadeas-hejnic linked an issue Sep 2, 2026 that may be closed by this pull request
@ynbot ynbot added the size/S label Sep 2, 2026
@tadeas-hejnic tadeas-hejnic self-assigned this Sep 2, 2026
Comment thread client/ayon_core/plugins/loader/delete_old_versions.py Outdated
@ynbot ynbot moved this to Review In Progress in PR reviewing Sep 2, 2026
Comment thread client/ayon_core/plugins/loader/delete_old_versions.py Outdated
@iLLiCiTiT iLLiCiTiT changed the title Enhancement: Delete Selected Versions action Delete Versions: Allow to delete selected versions Sep 2, 2026
@tadeas-hejnic tadeas-hejnic added the type: enhancement Improvement of existing functionality or minor addition label Sep 3, 2026
@tadeas-hejnic
tadeas-hejnic marked this pull request as ready for review September 3, 2026 09:43
Comment thread client/ayon_core/plugins/loader/delete_old_versions.py Outdated
Comment thread client/ayon_core/plugins/loader/delete_old_versions.py Outdated
@tadeas-hejnic

tadeas-hejnic commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author
image

Group label added. Ready to merge now? @iLLiCiTiT

Or should I put the Calculate versions size also to the group?

@iLLiCiTiT iLLiCiTiT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. @BigRoy can you take a quick look?

@github-project-automation github-project-automation Bot moved this from Review In Progress to Merge Requested in PR reviewing Sep 4, 2026

@BigRoy BigRoy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Clicking the entry works for me.

However for whatever reason it's not in a submenu inside the context menu for me:

Image

@iLLiCiTiT you may know why?
Would be nice if we can check that.

Tiny side note is that when you trigger it the version remains shown in the loader UI until you refresh. I wonder if there's a way to trigger a refresh to the loader itself. But maybe that's more of a problem to solve with the new browser at some point.

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

Labels

size/S type: enhancement Improvement of existing functionality or minor addition

Projects

Status: Merge Requested

Development

Successfully merging this pull request may close these issues.

YN-0686: Loader delete versions

4 participants