Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

General: OpenPype modules publish plugins are registered in host #3180

Merged
merged 4 commits into from
May 18, 2022

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented May 13, 2022

Brief description

Module's publish plugin paths are registered during installation of OpenPype plugins.

Description

Changed how module plugin paths are registered. Instead of using environment variables are plugin paths registered during installation of OpenPype plugins which is more suitable and possible dynamic.

Additional info

This approach gives more dynamic approach of registering publish plugins (e.g. does not require restart of tray when module change state). Can be extended in future to install plugin paths per project and more dynamically, which is not as easy now because modules' plugin paths are there from moment when openpype starts.

PR is based on #3179 where creator plugins are installed same way. I wanted to separate the PRs as this change should be fully tested.

Testing notes:

  1. All modules should register their publish plugins in all hosts
  2. The same must work for farm publishing, standalone publishing and webpublisher

@iLLiCiTiT iLLiCiTiT self-assigned this May 13, 2022
@iLLiCiTiT iLLiCiTiT added Bump Minor Pull requests that update a dependency file type: feature Larger, user affecting changes and completely new things labels May 13, 2022
Copy link
Member

@kalisp kalisp left a comment

Choose a reason for hiding this comment

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

Tested in AE,PS, Nuke, Standalone publisher

@iLLiCiTiT iLLiCiTiT merged commit f0ad33a into develop May 18, 2022
@mkolar mkolar deleted the feature/modules_plugins_register_in_host branch May 30, 2022 08:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Bump Minor Pull requests that update a dependency file type: feature Larger, user affecting changes and completely new things
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants