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

Ftrack: Integrate ftrack api fix #3044

Merged
merged 1 commit into from
Apr 8, 2022
Merged

Ftrack: Integrate ftrack api fix #3044

merged 1 commit into from
Apr 8, 2022

Conversation

iLLiCiTiT
Copy link
Member

Brief description

Fixed wrong looping over dictionary object.

Description

Loop over dictionary did not use items method. Bug was added with this PR.

Testing notes:

  1. Run any publish which will trigger integrate ftrack api and will cause that new asset type will be created
  • asset type name can be defined in project_settings/ftrack/publish/IntegrateFtrackInstance/family_mapping

@iLLiCiTiT iLLiCiTiT self-assigned this Apr 8, 2022
@iLLiCiTiT iLLiCiTiT added type: bug Something isn't working module: Ftrack labels Apr 8, 2022
@iLLiCiTiT iLLiCiTiT requested a review from 64qam April 8, 2022 15:54
@iLLiCiTiT iLLiCiTiT merged commit 21337d3 into develop Apr 8, 2022
@iLLiCiTiT iLLiCiTiT deleted the bugfix/integrate_ftrack_asset_type_dic branch April 8, 2022 16:08
@mkolar mkolar added this to the next milestone Apr 11, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@64qam 64qam 64qam approved these changes

Assignees

@iLLiCiTiT iLLiCiTiT

Labels
module: Ftrack type: bug Something isn't working
Projects
None yet
Milestone
next
Development

Successfully merging this pull request may close these issues.
3 participants
@iLLiCiTiT @64qam @mkolar