ignoring the list return vs the typical dictionary

Author: bandophahita

atlassian/jira.py

@@ -3368,17 +3368,17 @@ def get_all_available_screen_fields(self, screen_id: T_id) -> T_resp_json:
         url = f"{base_url}/{screen_id}/availableFields"
         return self.get(url)
 
-    def get_screen_tabs(self, screen_id: T_id) -> list:
+    def get_screen_tabs(self, screen_id: T_id) -> Optional[list]:
         """
         Get tabs for the screen id
         :param screen_id:
         :return:
         """
         base_url = self.resource_url("screens")
         url = f"{base_url}/{screen_id}/tabs"
-        return self.get(url)
+        return self.get(url)  # type: ignore[return-value]
 
-    def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> list:
+    def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> Optional[list]:
         """
         Get fields by the tab id and the screen id
         :param tab_id:
@@ -3387,20 +3387,20 @@ def get_screen_tab_fields(self, screen_id: T_id, tab_id: T_id) -> list:
         """
         base_url = self.resource_url("screens")
         url = f"{base_url}/{screen_id}/tabs/{tab_id}/fields"
-        return self.get(url)
+        return self.get(url)  # type: ignore[return-value]
 
     def get_all_screen_fields(self, screen_id: T_id) -> list:
         """
         Get all fields by screen id
         :param screen_id:
         :return:
         """
-        screen_tabs = self.get_screen_tabs(screen_id)
+        screen_tabs = self.get_screen_tabs(screen_id) or []
         fields: list = []
         for screen_tab in screen_tabs:
             tab_id = screen_tab["id"]
             if tab_id:
-                tab_fields = self.get_screen_tab_fields(screen_id=screen_id, tab_id=tab_id)
+                tab_fields = self.get_screen_tab_fields(screen_id=screen_id, tab_id=tab_id) or []
                 fields = fields + tab_fields
         return fields