Merge pull request #40 from fabiohaifa/ISSUE-34

chore: documentation adjustment

quickstart/README.md

@@ -98,7 +98,7 @@ Execute all these steps below to setup your evironment before running the Hands-
 
         ```powershell
         az config set extension.use_dynamic_install=yes_without_prompt
-
+        Connect-AzureAD
         ./quickstart/scripts/cloud-setup/Deploy-AzurePreReqs.ps1 -ConfigurationFile "quickstart/configs/cloud-setup/hol.json"
         ```
 

quickstart/scripts/modules/Azure.psm1

@@ -155,7 +155,14 @@ function AssignApplicationAdministratorAZRole
     }
 
     # Add the SP to role
-    Add-AzureADDirectoryRoleMember -ObjectId $role.ObjectId  -RefObjectId $ObjectId
+    try {
+        Add-AzureADDirectoryRoleMember -ObjectId $role.ObjectId  -RefObjectId $ObjectId
+        LogInfo -Message "Service Principal add into Role Application administrator with success!"
+    }
+    catch { 
+        LogInfo -Message "Service Principal already have Application administrator role."
+    }
+    
 }
 
 function AssignRoleIfNotExists