Code cleanup on ConnectionType (#1594)

* Renaming connection type so it becomes more meaningful

* Renaming back to void backwards compatibility issue

* Also reverting TenantAdmin enum

Author: KoenZomers

src/Commands/Base/PnPConnection.cs

@@ -89,11 +89,19 @@ internal PnPContext PnPContext
         /// </summary>
         public string ClientSecret { get; protected set; }
 
-        //public TelemetryClient TelemetryClient { get; set; }
+        /// <summary>
+        /// Azure Application Insights instance to provide telemetry
+        /// </summary>
         public ApplicationInsights ApplicationInsights { get; set; }
 
+        /// <summary>
+        /// Url of the SharePoint Online site to connect to
+        /// </summary>
         public string Url { get; protected set; }
 
+        /// <summary>
+        /// Url of the SharePoint Online Admin center to use. If omitted, it will try to automatically determine this.
+        /// </summary>
         public string TenantAdminUrl { get; protected set; }
 
         /// <summary>
@@ -106,6 +114,9 @@ internal PnPContext PnPContext
         /// </summary>
         public bool DeleteCertificateFromCacheOnDisconnect { get; internal set; }
 
+        /// <summary>
+        /// ClientContext to use to execute Client Side Object Model (CSOM) requests
+        /// </summary>
         public ClientContext Context { get; set; }
 
         /// <summary>
@@ -117,6 +128,10 @@ internal PnPContext PnPContext
         /// Defines if this is a managed identity connection for use in cloud shell
         /// </summary>
         internal bool ManagedIdentity { get; set; }
+
+        /// <summary>
+        /// Type of Azure cloud to connect to
+        /// </summary>
         public AzureEnvironment AzureEnvironment { get; set; } = AzureEnvironment.Production;
 
         private string _graphEndPoint;

src/Commands/Enums/ConnectionType.cs

@@ -1,8 +1,18 @@
 namespace PnP.PowerShell.Commands.Enums
 {
+    /// <summary>
+    /// Defines to what type of site collection in SharePoint Online a connection has been made
+    /// </summary>
     public enum ConnectionType
     {
+        /// <summary>
+        /// Normal SharePoint Online site collection
+        /// </summary>
         O365 = 1,
+
+        /// <summary>
+        /// SharePoint Online Admin site collection
+        /// </summary>
         TenantAdmin = 2,
     }
 }