Make shotgun login dialog visible and clickable

openpype/modules/shotgun/hooks/post_shotgun_changes.py

@@ -1,5 +1,3 @@
-import os
-
 from openpype.lib import PostLaunchHook
 
 

openpype/modules/shotgun/shotgun_module.py

@@ -1,6 +1,6 @@
 import os
 from typing import Optional, Dict, AnyStr, Any
-from Qt import QtWidgets
+
 from openpype.modules import PypeModule, ITrayModule, IPluginPaths, ILaunchHookPaths
 from openpype.modules.shotgun.tray.shotgun_tray import ShotgunTrayWrapper
 
@@ -44,4 +44,3 @@ def tray_exit(self, *args, **kwargs):
 
  def tray_menu(self, tray_menu):
  return self.tray_wrapper.tray_menu(tray_menu)
-

openpype/modules/shotgun/tray/credential_dialog.py

@@ -1,11 +1,8 @@
-import os
 from typing import Any
 
-import requests
+from Qt import QtCore, QtWidgets
+
 from openpype import style
-from openpype.modules.ftrack.lib import credentials
-from openpype import resources
-from Qt import QtCore, QtGui, QtWidgets
 
 
 class CredentialsDialog(QtWidgets.QDialog):

openpype/modules/shotgun/tray/shotgun_tray.py

@@ -3,6 +3,7 @@
 from openpype.modules.shotgun.tray.credential_dialog import CredentialsDialog
 from Qt import QtWidgets
 
+
 class ShotgunTrayWrapper:
  module: Any
  credentials_dialog: CredentialsDialog
@@ -11,6 +12,9 @@ def __init__(self, module) -> None:
  self.module = module
  self.credentials_dialog = CredentialsDialog(module)
 
+ def show_connect_dialog(self):
+ self.show_credential_dialog()
+
  def show_credential_dialog(self):
  self.credentials_dialog.show()
  self.credentials_dialog.activateWindow()
@@ -19,9 +23,12 @@ def show_credential_dialog(self):
  def tray_menu(self, tray_menu):
  print(type(tray_menu))
  menu = QtWidgets.QMenu("Shotgun", tray_menu)
+ show_connect_action = QtWidgets.QAction("Connect to Shotgun", menu)
+ show_connect_action.triggered.connect(self.show_connect_dialog)
+ menu.addAction(show_connect_action)
+
  tray_menu.addMenu(menu)
 
  def validate(self):
  self.show_credential_dialog()
  return True
-

setup.cfg

@@ -1,7 +1,7 @@
 [flake8]
 # ignore = D203
 ignore = BLK100, W504, W503
-max-line-length = 79
+max-line-length = 88
 exclude =
  .git,
  __pycache__,
@@ -26,4 +26,4 @@ omit = /tests
 directory = ./coverage
 
 [tool:pytest]
-norecursedirs = repos/* openpype/modules/ftrack/*
+norecursedirs = repos/* openpype/modules/ftrack/*