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.

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

Refactor ayon_sitesync/providers/lib.py #46

Open
1 task done
kalisp opened this issue Dec 10, 2024 · 0 comments
Open
1 task done

Refactor ayon_sitesync/providers/lib.py #46

kalisp opened this issue Dec 10, 2024 · 0 comments
Assignees
Labels
type: enhancement Improvement of existing functionality or minor addition

Comments

@kalisp
Copy link
Member

kalisp commented Dec 10, 2024

Is there an existing issue for this?

  • I have searched the existing issues and added correct labels.

Please describe the feature you have in mind and explain what the current shortcomings are?

This file directly imports all dependencies even if they might not be needed. It could happen if provider is not selected in any site or if SiteSync addon should be deleted.

Suggested implementation?

Move imports only to class itself when _get_creator_info will not find that provider.

Describe alternatives you've considered:

No response

@kalisp kalisp added the type: enhancement Improvement of existing functionality or minor addition label Dec 10, 2024
@kalisp kalisp self-assigned this Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

No branches or pull requests

1 participant