ynput / ayon-ftrack

Apache License 2.0
3 stars 9 forks source link

Event handlers: Status task to version settings #120

Closed iLLiCiTiT closed 2 months ago

iLLiCiTiT commented 2 months ago

Changelog Description

Fix attribute name in settings model.

Additional info

Status to version settings model has correct attribute name asset_types_filter instead of asset_types_to_skip.

Testing notes:

  1. Task to version status propagation works.
iLLiCiTiT commented 2 months ago

DISCLAIMER: As I'm thinking about it, this may actually break stuff because of default settings Version > Task and Task > Version status sync. We should probably (at both places) have Allow list and Deny list, so it can work both ways.

iLLiCiTiT commented 2 months ago

Changed the PR to simple fix similar to https://github.com/ynput/ayon-ftrack/pull/121 and created new PR https://github.com/ynput/ayon-ftrack/pull/123 with enahnced logic.