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

Tray: Connection timeout and update validations #28

Merged
merged 5 commits into from
Feb 22, 2024

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Feb 12, 2024

Changelog Description

Tray tool can validate if is required update of AYON bundle and asks to relogin if user credentials are not valid.

Additional info

The validation happens once per 5 minutes. The credentials check happens withing it. It is still not possible to re-login when inside host and the validation happens only once a 5 minutes, but better than nothing. The update dialog is shown only in production or staging mode.

Possible enhancements in this PR

Maybe we should show information dialog "Your credentials are not valid anymore. Please close all applications and login again." before the login dialog is showed.

Testing notes:

Updates detection

  1. Start tray in production
  2. Change production bundle on server
  3. Wait 5 minutes
  4. An dialog with restart request should show up

Relogin

  1. Start tray
  2. Shotdown your server
  3. After 5 minutes a login dialog should be shown

ayon-core variant of ynput/OpenPype#6194

Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

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

Works for both cases / occasions... without any issues.

When Ayon Bundle changes during running Ayon client:

Screenshot 2024-02-21 105811

When Ayon server being down:

Screenshot 2024-02-21 110603

@iLLiCiTiT iLLiCiTiT merged commit d563437 into develop Feb 22, 2024
2 checks passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/OP-7630_DCC-connection-timeout branch February 22, 2024 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants