OpenPype icon indicating copy to clipboard operation
OpenPype copied to clipboard

TrayPublisher: Removed from experimental tools

Open iLLiCiTiT opened this issue 2 years ago • 3 comments

Brief description

Tray publisher is added to tray menu by default and artist don't have to enable the experimental option.

Description

Tray Publisher is now in ready to use state and should work at least the same way as standalone publisher did. We should collect as much input from artists as possible thus the tool is removed from experimental tools and is added as available feature in tray.

Additional information

Changed label to Tray Publish so it is possible to differentiate in standalone publisher and tray publisher.

Testing notes:

  1. Tray publisher is visible in tray menu without enabling it in local settings
  2. Tray publisher is not visible in local settings

iLLiCiTiT avatar Aug 15 '22 12:08 iLLiCiTiT

This is confusing. If we take it out of experimental, we need to at the same time hide the old one and make that optional. Two publishers at the same time is a road to hell I'm afraid. Do we now have 1-1 functionality match?

mkolar avatar Aug 17 '22 12:08 mkolar

Do we now have 1-1 functionality match?

There is not 100% but at the same time we should add an easy access to use tray publisher so somebody in production can test what we have there now.

We can add standalone publisher and tray publisher into module settings so they can be enabled/disabled. Also we can change label of standalone publisher action.

iLLiCiTiT avatar Aug 17 '22 14:08 iLLiCiTiT

It is not clear what should happen next.

current Publish to be renamed and hidden? (Legacy Publish and create settings to put it back on, or?)

current Tray Publish renamed to Publish (or keep Tray Publish? or?)

m-u-r-p-h-y avatar Sep 13 '22 12:09 m-u-r-p-h-y

I would remove the old publisher and keep the new one just called Publisher (to keep in liner with the loader and launcher rather than load and launch). I agree that we should keep the option to turn the old publisher back on somewhere in the settings. In which case the old should be called Publisher (legacy)

mkolar avatar Nov 07 '22 19:11 mkolar

Standalone publisher is disabled by default (can be enabled) and has label Publisher (legacy). Tray publisher has label Publisher.

iLLiCiTiT avatar Nov 10 '22 10:11 iLLiCiTiT