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

Ftrack: Removed ftrack interface #2049

Merged
merged 5 commits into from
Sep 22, 2021
Merged

Ftrack: Removed ftrack interface #2049

merged 5 commits into from
Sep 22, 2021

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Sep 20, 2021 •
edited
Loading

Goal

  • ftrack module don't use interface to define which modules want to add it's path to ftrack event handlers

Changes

  • removed ftrack's IFtrackEventHandlerPaths and just check for existence of get_ftrack_event_handler_paths method

@iLLiCiTiT iLLiCiTiT mentioned this pull request Sep 20, 2021
Closed
4 tasks
@iLLiCiTiT iLLiCiTiT self-assigned this Sep 20, 2021
@iLLiCiTiT iLLiCiTiT added module: Ftrack type: enhancement Enhancements to existing functionality labels Sep 20, 2021
@iLLiCiTiT iLLiCiTiT merged commit 1b05150 into develop Sep 22, 2021
@iLLiCiTiT iLLiCiTiT deleted the feature/ftrack_without_interface branch December 13, 2021 15:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kalisp kalisp kalisp approved these changes

@antirotor antirotor Awaiting requested review from antirotor

Assignees

@iLLiCiTiT iLLiCiTiT

Labels
module: Ftrack type: enhancement Enhancements to existing functionality
Projects
None yet
Milestone
No milestone
Development

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