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

AYON to ftrack sync: Sync AYON changes to ftrack #166

Merged
merged 70 commits into from
Dec 9, 2024

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Nov 11, 2024

Changelog Description

Synchronization of AYON changes to ftrack.

Additional review information

This PR combination of multiple other PRs.

  • New service 'transmitter' in ftrack addon which listens to AYON events and handles sync to ftrack.
  • Synchronization of assignees between tasks if ftrack id is filled in AYON.
  • Synchronization of statuses from AYON to ftrack.
  • Synchronization of attributes from AYON to ftrack.
  • Simple synchronization of comment from AYON to ftrack. We don't capture removed comments in AYON (future enhancement).

Testing notes:

  1. Clone repository and go to ./service_tools.
  2. Run ./manage.ps1 install on windows or make install on linux/mac.
  3. Run ./manage.ps1 services on windows or make services on linux/mac (You can run just transmitter instead of services).
  4. Changes in AYON should propagate to ftrack and changes in ftrack should propagate to AYON.

Resolves #133

@iLLiCiTiT iLLiCiTiT self-assigned this Nov 11, 2024
@iLLiCiTiT iLLiCiTiT added the type: feature Adding something new and exciting to the product label Nov 11, 2024
…ayon-to-ftrack

AYON to ftrack sync: Sync assignments on task
…-ayon-to-ftrack

AYON to ftrack sync: Sync statuses
…-from-ayon-to-ftrack

AYON to ftrack sync: Sync attribute changes
@iLLiCiTiT iLLiCiTiT requested a review from 64qam December 5, 2024 14:19
@iLLiCiTiT iLLiCiTiT marked this pull request as ready for review December 5, 2024 14:19
Copy link
Member

@64qam 64qam left a comment

Choose a reason for hiding this comment

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

LGTM @MilaKudr could you please also test it?

@MilaKudr
Copy link

MilaKudr commented Dec 6, 2024

In many cases it works, but for some tasks/shots I get these errors during Ayon to ftrack sync:
File "C:\Users\Mila\code\ayon-ftrack\service_tools\venv\lib\site-packages\ftrack_api\session.py", line 1687, in call
raise ftrack_api.exception.ServerError(error_message)
ftrack_api.exception.ServerError: Server reported error: ParseError(No attribute 'parent_id exists for schema 'Asset'.)

@iLLiCiTiT
Copy link
Member Author

In many cases it works, but for some tasks/shots I get these errors during Ayon to ftrack sync:
File "C:\Users\Mila\code\ayon-ftrack\service_tools\venv\lib\site-packages\ftrack_api\session.py", line 1687, in call
raise ftrack_api.exception.ServerError(error_message)
ftrack_api.exception.ServerError: Server reported error: ParseError(No attribute 'parent_id exists for schema 'Asset'.)

Fixed, try again.

@iLLiCiTiT iLLiCiTiT merged commit e914711 into develop Dec 9, 2024
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the feature/133-ftrack-2ways-sync branch December 9, 2024 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump minor type: feature Adding something new and exciting to the product
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ftrack 2ways Sync
3 participants