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

Update custom ftrack session attributes #1202

Merged
merged 2 commits into from
Mar 30, 2021

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Mar 29, 2021

Issue

Ftrack api's Session has new attribute which is not available in our modified Session so it is not possible to start ftrack server with newer versions of ftrack api.

Changes

  • new ftrack api session has new attribute request_timeout which is not available in modified session class
    • few updates in custom session

@iLLiCiTiT iLLiCiTiT self-assigned this Mar 29, 2021
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Traceback (most recent call last):
Traceback (most recent call last):
  File "/home/linters/.local/bin/flake8", line 5, in 
    from flake8.main.cli import main
ModuleNotFoundError: No module named 'flake8'

@iLLiCiTiT iLLiCiTiT requested a review from a team March 29, 2021 09:54
@iLLiCiTiT iLLiCiTiT added 2x-dev type: bug Something isn't working labels Mar 29, 2021
Copy link

@hound hound bot left a comment

Choose a reason for hiding this comment

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

Some files could not be reviewed due to errors:

Traceback (most recent call last):
Traceback (most recent call last):
  File "/home/linters/.local/bin/flake8", line 5, in 
    from flake8.main.cli import main
ModuleNotFoundError: No module named 'flake8'

@mkolar mkolar added this to the 2.16.1 milestone Mar 29, 2021
@mkolar mkolar merged commit 7909364 into 2.x/main Mar 30, 2021
@mkolar mkolar deleted the hotfix/update_ftrack_session_attribute branch April 5, 2021 21:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants