Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use AYONAddon base class #22

Merged
merged 5 commits into from
Feb 9, 2024
Merged

Use AYONAddon base class #22

merged 5 commits into from
Feb 9, 2024

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Use AYONAddon base class for all possible addons.

Additional info

Some of the addons still must use OpenPypeModule because of settings, but that is minority at this moment. Renamed library loader action to just loader action.

Testing notes:

  1. All addons and hosts should load correctly.

MustafaJafar
MustafaJafar approved these changes Feb 9, 2024
View reviewed changes
Copy link
Contributor

@MustafaJafar MustafaJafar left a comment

Choose a reason for hiding this comment

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

Houdini works as expected.
It loads fine.

@iLLiCiTiT iLLiCiTiT merged commit b32f3c5 into develop Feb 9, 2024
2 of 5 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/use-ayon-addon-base-class branch February 9, 2024 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MustafaJafar MustafaJafar MustafaJafar approved these changes

@simonebarbieri simonebarbieri Awaiting requested review from simonebarbieri

@kalisp kalisp Awaiting requested review from kalisp

@antirotor antirotor Awaiting requested review from antirotor

@jakubjezek001 jakubjezek001 Awaiting requested review from jakubjezek001

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@iLLiCiTiT @MustafaJafar