Skip to content

Enhancement: Allow server options for automatic resolution of "Local cache of version exists" dialog - #139

Open
Sasbom wants to merge 5 commits into
ynput:developfrom
Sasbom:enhancement/allow-default-on-cache-conflict
Open

Enhancement: Allow server options for automatic resolution of "Local cache of version exists" dialog#139
Sasbom wants to merge 5 commits into
ynput:developfrom
Sasbom:enhancement/allow-default-on-cache-conflict

Conversation

@Sasbom

@Sasbom Sasbom commented Jul 17, 2026

Copy link
Copy Markdown

This PR adds a server option for automatically resolving the dialog that pops up when a newer, locally cached file is present.
This allows TDs setting up the plugin to be strict about there source of truth, instead of leaving it to the artist.
The PR was set up following a discussion within the community about this dialog's choice leaving a workflow vulnerable to loss of work.

The new options are disabled by default, and without enabling the "Force cache conflict source" option, in "Cache conflict resolution", the dialog will still pop up.

Thanks for your consideration,

Sas van Gulik

Changelog Description

This PR adds a server-side toggle + enum with corresponding options found in the "Local cache of version exists" dialog's buttons.
It allows the user to set a default strategy if that is wanted.
A server for fetching server options have been added to the clients api/lib.py.

Additional review information

I'm still not quite sure about the names of the options. That could be clearer, but I think this gets the points across.

Testing notes:

  1. Include the plugin built with this branch in a bundle
  2. Set "Cache conflict resolution" settings.
  3. Launch Harmony with a known project that would give the "Local cache of version exists" dialog on startup, and see if expected changes are reflected.

@kalisp
kalisp requested review from BigRoy, iLLiCiTiT and kalisp July 17, 2026 10:57
Comment thread client/ayon_harmony/api/lib.py Outdated
Comment thread client/ayon_harmony/api/lib.py Outdated
Comment thread client/ayon_harmony/api/lib.py Outdated
Comment thread client/ayon_harmony/api/lib.py Outdated
Comment thread client/ayon_harmony/api/lib.py
Comment thread client/ayon_harmony/api/lib.py
Comment thread server/settings/cache_default.py Outdated
@Sasbom
Sasbom requested a review from iLLiCiTiT July 22, 2026 09:28
@kalisp

kalisp commented Aug 4, 2026

Copy link
Copy Markdown
Member

Could you please resolve conversation to mark that next round of testing/review should happen?

@Sasbom

Sasbom commented Aug 4, 2026

Copy link
Copy Markdown
Author

Could you please resolve conversation to mark that next round of testing/review should happen?

Done! Excuse my etiquette. Thanks for the reminder.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants