Misc/improvement

.pylintrc

@@ -421,6 +421,6 @@ valid-metaclass-classmethod-first-arg=mcs
 
 # Exceptions that will emit a warning when being caught. Defaults to
 # "Exception"
-overgeneral-exceptions=StandardError,
-                       Exception,
-                       BaseException
+overgeneral-exceptions=builtins.StandardError,
+                       builtins.Exception,
+                       builtins.BaseException

src/dfcx_scrapi/core/agents.py

@@ -418,8 +418,8 @@ def restore_agent(self, agent_id: str, gcs_bucket_uri: str) -> str:
         Args:
           agent_id: CX Agent ID string in the following format
             projects/<PROJECT ID>/locations/<LOCATION ID>/agents/<AGENT ID>
-          gcs_bucket_uri: The Google Cloud Storage bucket/filepath to export the
-            agent to in the following format:
+          gcs_bucket_uri: The Google Cloud Storage bucket/filepath to restore
+            the agent from in the following format:
               `gs://<bucket-name>/<object-name>`
 
         Returns:

src/dfcx_scrapi/core/changelogs.py

@@ -225,7 +225,7 @@ def changelogs_to_dataframe(
                 ],
             )
 
-            df = df.append(log_data)
+            df = pd.concat([df, log_data], ignore_index=True)
 
         df = df.reset_index(drop=True)
 

src/dfcx_scrapi/core/entity_types.py

@@ -422,7 +422,7 @@ def update_entity_type(
 
     @scrapi_base.api_call_counter_decorator
     def delete_entity_type(self, entity_id: str = None, obj=None) -> None:
-        """Deletes a single Entity Type resouce object.
+        """Deletes a single Entity Type resource object.
 
         Args:
           entity_id: the formatted CX Entity ID to delete

src/dfcx_scrapi/tools/copy_util.py

@@ -134,36 +134,22 @@ def _convert_entry_webhooks(page_object, webhooks_map):
     def __convert_tr_target_page(
         trans_route, pages_map, convert_type=None, flows_map=None, flow=None
     ):
+        special_pages = [
+            "END_FLOW", "END_SESSION",
+            "CURRENT_PAGE", "PREVIOUS_PAGE", "START_PAGE"
+        ]
 
         if convert_type == "source":
-            if trans_route.target_page.split("/")[-1] == "END_FLOW":
-                trans_route.target_page = "END_FLOW"
-            elif trans_route.target_page.split("/")[-1] == "END_SESSION":
-                trans_route.target_page = "END_SESSION"
-            elif trans_route.target_page.split("/")[-1] == "CURRENT_PAGE":
-                trans_route.target_page = "CURRENT_PAGE"
-            elif trans_route.target_page.split("/")[-1] == "PREVIOUS_PAGE":
-                trans_route.target_page = "PREVIOUS_PAGE"
-            elif trans_route.target_page.split("/")[-1] == "START_PAGE":
-                trans_route.target_page = "START_PAGE"
+            last_part = trans_route.target_page.split("/")[-1]
+            if last_part in special_pages:
+                trans_route.target_page = last_part
             else:
                 trans_route.target_page = pages_map[trans_route.target_page]
 
         elif convert_type == "destination":
-            if trans_route.target_page == "END_FLOW":
-                trans_route.target_page = flows_map[flow] + "/pages/END_FLOW"
-            elif trans_route.target_page == "END_SESSION":
-                trans_route.target_page = flows_map[flow] + "/pages/END_SESSION"
-            elif trans_route.target_page == "CURRENT_PAGE":
-                trans_route.target_page = (
-                    flows_map[flow] + "/pages/CURRENT_PAGE"
-                )
-            elif trans_route.target_page == "PREVIOUS_PAGE":
-                trans_route.target_page = (
-                    flows_map[flow] + "/pages/PREVIOUS_PAGE"
-                )
-            elif trans_route.target_page == "START_PAGE":
-                trans_route.target_page = flows_map[flow] + "/pages/START_PAGE"
+            if trans_route.target_page in special_pages:
+                new_page = f"{flows_map[flow]}/pages/{trans_route.target_page}"
+                trans_route.target_page = new_page
             else:
                 trans_route.target_page = pages_map[trans_route.target_page]