Usability improvements and bug fix

Fixed bug where the last net-new custom performance class was not created. In this scenario, any VPN uplink prefs that relied on it could not be restored unless the user ran the script a second time.

Author: TKIPisalegacycipher

notebooks/Uplink preference backup and restore/merakiUplinkPreferenceRestore.ipynb

@@ -80,7 +80,7 @@
     "user_data = dict()\n",
     "\n",
     "# Set the filename to use for the backup workbook\n",
-    "WORKBOOK_FILENAME = 'exampleBackups/downloaded_rules_workbook_2020-12-03 live demo workbook.xlsx'"
+    "WORKBOOK_FILENAME = 'exampleBackups/downloaded_rules_workbook_2020-12-01 backup with wan and vpn uplink prefs and cpcs.xlsx'"
    ]
   },
   {
@@ -676,6 +676,9 @@
     "\n",
     "\t# If there are any orphans left, they have not matched by ID or name. Create them new.\n",
     "\tif len(list_of_orphan_loaded_performance_classes):\n",
+    "\t\tprint(f'{len(list_of_orphan_loaded_performance_classes)} new custom performance classes need to be created:')\n",
+    "\t\tprint(f'{list_of_orphan_loaded_performance_classes}\\n')\n",
+    "\n",
     "\t\tfor orphan_loaded_performance_class in list_of_orphan_loaded_performance_classes:\n",
     "\t\t\t# Re-create the loaded class from the backup and get its new ID\n",
     "\t\t\t# We'll also add the old and new IDs to the reference object we've created for this purpose\n",
@@ -686,6 +689,7 @@
     "\t\t\t\tname=orphan_loaded_performance_class['name'],\n",
     "\t\t\t\tmaxLossPercentage=orphan_loaded_performance_class['maxLossPercentage']\n",
     "\t\t\t)\n",
+    "\t\t\tprint(f'Created new custom performance class {new_performance_class} from the backup\\'s {orphan_loaded_performance_class}.\\n')\n",
     "\t\t\t\n",
     "\t\t\t# Add it to the name-only matches list, list_of_name_matches\n",
     "\t\t\tlist_of_id_updates.append(\n",
@@ -695,9 +699,8 @@
     "\t\t\t\t}\n",
     "\t\t\t)\n",
     "\n",
-    "\t\t\t# Remove it from the list of orphans\n",
-    "\t\t\tlist_of_orphan_loaded_performance_classes.remove(orphan_loaded_performance_class)\n",
-    "\n",
+    "\tprint('These IDs from the backup will need to be updated:')\n",
+    "\tprint(f'{list_of_id_updates}\\n\\n')\n",
     "\t# Return the list of updated IDs in key-value pairs for later processing\n",
     "\treturn(list_of_id_updates)"
    ]