From 2eb175ec6f526819d382e7a5f0aec950bc478e0e Mon Sep 17 00:00:00 2001
From: Geoffrey Dudics <stcyryn@hotmail.com>
Date: Thu, 16 Apr 2020 23:50:10 -0400
Subject: [PATCH 01/16] Fixed some CS1572, 1573, 1574 and 1591 warnings

---
 .../INavigationServiceExtensions.cs           |  6 ++
 .../Navigation/PageNavigationService.cs       |  1 -
 .../INavigationServiceExtensions.cs           |  1 +
 .../Prism.Unity.Forms/PrismApplication.cs     |  3 +
 src/Prism.Core/Common/ParametersBase.cs       | 62 ++++++++++++++++++-
 src/Prism.Core/Modularity/IModuleInfo.cs      | 25 +++++++-
 src/Prism.Core/Modularity/IModuleInfoGroup.cs | 10 +++
 .../Legacy/DryIocExtensions.cs                |  1 +
 8 files changed, 106 insertions(+), 3 deletions(-)

diff --git a/src/Forms/Prism.Forms/Navigation/INavigationServiceExtensions.cs b/src/Forms/Prism.Forms/Navigation/INavigationServiceExtensions.cs
index 547c67fdf4..bb07b0fe66 100644
--- a/src/Forms/Prism.Forms/Navigation/INavigationServiceExtensions.cs
+++ b/src/Forms/Prism.Forms/Navigation/INavigationServiceExtensions.cs
@@ -13,6 +13,7 @@ public static class INavigationServiceExtensions
         /// <summary>
         /// Navigates to the most recent entry in the back navigation history by popping the calling Page off the navigation stack.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <param name="useModalNavigation">If <c>true</c> uses PopModalAsync, if <c>false</c> uses PopAsync</param>
         /// <param name="animated">If <c>true</c> the transition is animated, if <c>false</c> there is no animation on transition.</param>
@@ -37,6 +38,7 @@ public static Task<INavigationResult> GoBackToRootAsync(this INavigationService
         /// <summary>
         /// Initiates navigation to the target specified by the <paramref name="name"/>.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="name">The name of the target to navigate to.</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <param name="useModalNavigation">If <c>true</c> uses PushModalAsync, if <c>false</c> uses PushAsync</param>
@@ -50,6 +52,7 @@ public static Task<INavigationResult> NavigateAsync(this INavigationService navi
         /// <summary>
         /// Initiates navigation to the target specified by the <paramref name="uri"/>.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="uri">The Uri to navigate to</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <param name="useModalNavigation">If <c>true</c> uses PopModalAsync, if <c>false</c> uses PopAsync</param>
@@ -87,6 +90,7 @@ public static string GetNavigationUriPath(this INavigationService navigationServ
         /// <summary>
         /// Navigates to the most recent entry in the back navigation history by popping the calling Page off the navigation stack.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <returns><see cref="INavigationResult"/> indicating whether the request was successful or if there was an encountered <see cref="Exception"/>.</returns>
         public static Task<INavigationResult> GoBackAsync(this INavigationService navigationService, params (string Key, object Value)[] parameters)
@@ -97,6 +101,7 @@ public static Task<INavigationResult> GoBackAsync(this INavigationService naviga
         /// <summary>
         /// Initiates navigation to the target specified by the <paramref name="name"/>.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="name">The Uri to navigate to</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <returns><see cref="INavigationResult"/> indicating whether the request was successful or if there was an encountered <see cref="Exception"/>.</returns>
@@ -112,6 +117,7 @@ public static Task<INavigationResult> NavigateAsync(this INavigationService navi
         /// <summary>
         /// Initiates navigation to the target specified by the <paramref name="uri"/>.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="uri">The Uri to navigate to</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <returns><see cref="INavigationResult"/> indicating whether the request was successful or if there was an encountered <see cref="Exception"/>.</returns>
diff --git a/src/Forms/Prism.Forms/Navigation/PageNavigationService.cs b/src/Forms/Prism.Forms/Navigation/PageNavigationService.cs
index d1cc565f09..9ad8ab8d73 100644
--- a/src/Forms/Prism.Forms/Navigation/PageNavigationService.cs
+++ b/src/Forms/Prism.Forms/Navigation/PageNavigationService.cs
@@ -173,7 +173,6 @@ Task<INavigationResult> IPlatformNavigationService.GoBackToRootAsync(INavigation
         /// <summary>
         /// When navigating inside a NavigationPage: Pops all but the root Page off the navigation stack
         /// </summary>
-        /// <param name="navigationService">The INavigatinService instance</param>
         /// <param name="parameters">The navigation parameters</param>
         /// <remarks>Only works when called from a View within a NavigationPage</remarks>
         protected async virtual Task<INavigationResult> GoBackToRootInternal(INavigationParameters parameters)
diff --git a/src/Forms/Prism.Forms/Navigation/TabbedPages/INavigationServiceExtensions.cs b/src/Forms/Prism.Forms/Navigation/TabbedPages/INavigationServiceExtensions.cs
index 39c30d03b9..4140907809 100644
--- a/src/Forms/Prism.Forms/Navigation/TabbedPages/INavigationServiceExtensions.cs
+++ b/src/Forms/Prism.Forms/Navigation/TabbedPages/INavigationServiceExtensions.cs
@@ -10,6 +10,7 @@ public static class INavigationServiceExtensions
         /// <summary>
         /// Selects a Tab of the TabbedPage parent.
         /// </summary>
+        /// <param name="navigationService">Service for handling navigation between views</param>
         /// <param name="name">The name of the tab to select</param>
         /// <param name="parameters">The navigation parameters</param>
         public static async Task<INavigationResult> SelectTabAsync(this INavigationService navigationService, string name, INavigationParameters parameters = null)
diff --git a/src/Forms/Prism.Unity.Forms/PrismApplication.cs b/src/Forms/Prism.Unity.Forms/PrismApplication.cs
index fa4167ec02..ed5e7e80bd 100644
--- a/src/Forms/Prism.Unity.Forms/PrismApplication.cs
+++ b/src/Forms/Prism.Unity.Forms/PrismApplication.cs
@@ -4,6 +4,9 @@
 [assembly: Xamarin.Forms.XmlnsDefinition("http://prismlibrary.com", "Prism.Unity")]
 namespace Prism.Unity
 {
+    /// <summary>
+    /// Base class for PrismApplication
+    /// </summary>
     public abstract class PrismApplication : PrismApplicationBase
     {
         /// <summary>
diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index d2a377f515..d720aa870c 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -7,14 +7,24 @@
 
 namespace Prism.Common
 {
+    /// <summary>
+    /// Base class for Navigation parameters
+    /// </summary>
     public abstract class ParametersBase : IEnumerable<KeyValuePair<string, object>>
     {
         private readonly List<KeyValuePair<string, object>> _entries = new List<KeyValuePair<string, object>>();
 
+        /// <summary>
+        /// Default constructor
+        /// </summary>
         protected ParametersBase()
         {
         }
 
+        /// <summary>
+        /// Constructs a list of parameters
+        /// </summary>
+        /// <param name="query">Query string to be parsed</param>
         protected ParametersBase(string query)
         {
             if (!string.IsNullOrWhiteSpace(query))
@@ -58,6 +68,12 @@ protected ParametersBase(string query)
             }
         }
 
+        /// <summary>
+        /// Searches Parameter collection and returns value if Collection contains key.
+        /// Otherswise returns null.
+        /// </summary>
+        /// <param name="key"></param>
+        /// <returns></returns>
         public object this[string key]
         {
             get
@@ -74,32 +90,72 @@ public object this[string key]
             }
         }
 
+        /// <summary>
+        /// The count, or number, of parameters in collection
+        /// </summary>
         public int Count => _entries.Count;
 
+        /// <summary>
+        /// Returns an IEnumerable of the Keys in the collection
+        /// </summary>
         public IEnumerable<string> Keys => 
             _entries.Select(x => x.Key);
 
+        /// <summary>
+        /// Adds the KeyValuePair to the Collection
+        /// </summary>
+        /// <param name="key"></param>
+        /// <param name="value"></param>
         public void Add(string key, object value) =>
             _entries.Add(new KeyValuePair<string, object>(key, value));
 
+        /// <summary>
+        /// Checks collection for presense of key
+        /// </summary>
+        /// <param name="key"></param>
+        /// <returns>True if key exists; else returns false.</returns>
         public bool ContainsKey(string key) =>
             _entries.ContainsKey(key);
 
+        /// <summary>
+        /// Gets an enumerator for the KeyValuePairs in parameter collection
+        /// </summary>
+        /// <returns>Enumerator</returns>
         public IEnumerator<KeyValuePair<string, object>> GetEnumerator() =>
             _entries.GetEnumerator();
-
+        /// <summary>
+        /// Returns the value of the member referenced by key
+        /// </summary>
+        /// <typeparam name="T">The type of object to be returned</typeparam>
+        /// <param name="key"></param>
+        /// <returns></returns>
         public T GetValue<T>(string key) => 
             _entries.GetValue<T>(key);
 
+        /// <summary>
+        /// Returns an IEnumerable of all parameters 
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="key"></param>
         public IEnumerable<T> GetValues<T>(string key) =>
             _entries.GetValues<T>(key);
 
+        /// <summary>
+        /// Checks to see if the parameter collection contains the value
+        /// </summary>
+        /// <typeparam name="T"></typeparam>
+        /// <param name="key"></param>
+        /// <param name="value">Value of the returned parameter if it exists</param>
         public bool TryGetValue<T>(string key, out T value) =>
             _entries.TryGetValue(key, out value);
 
         IEnumerator IEnumerable.GetEnumerator() =>
             GetEnumerator();
 
+        /// <summary>
+        /// Converts parameter collection to a parameter string
+        /// </summary>
+        /// <returns></returns>
         public override string ToString()
         {
             var queryBuilder = new StringBuilder();
@@ -129,6 +185,10 @@ public override string ToString()
             return queryBuilder.ToString();
         }
 
+        /// <summary>
+        /// Adds a collection of parameters to the local parameter list
+        /// </summary>
+        /// <param name="parameters"></param>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public void FromParameters(IEnumerable<KeyValuePair<string, object>> parameters) =>
             _entries.AddRange(parameters);
diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index 9ade53fe96..3896b210e8 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -2,13 +2,36 @@
 
 namespace Prism.Modularity
 {
-    public interface IModuleInfo : IModuleCatalogItem
+    /// <summary>
+    /// Set of properties for each Module
+    /// </summary>
+public interface IModuleInfo : IModuleCatalogItem
     {
+        /// <summary>
+        /// The modules this instance depends on.
+        /// </summary>
         Collection<string> DependsOn { get; set; }
+        /// <summary>
+        /// When Prism should Initialize the module
+        /// <see cref="InitializationMode"/>
+        /// </summary>
         InitializationMode InitializationMode { get; set; }
+        /// <summary>
+        /// The name of the module
+        /// </summary>
         string ModuleName { get; set; }
+        /// <summary>
+        /// The module's type
+        /// </summary>
         string ModuleType { get; set; }
+        /// <summary>
+        /// A string ref for the module for registration/resolution
+        /// </summary>
         string Ref { get; set; }
+        /// <summary>
+        /// An enum that lists the modules state
+        /// <see cref="ModuleState"/>
+        /// </summary>
         ModuleState State { get; set; }
     }
 }
\ No newline at end of file
diff --git a/src/Prism.Core/Modularity/IModuleInfoGroup.cs b/src/Prism.Core/Modularity/IModuleInfoGroup.cs
index c4395df52d..3e021d4ff1 100644
--- a/src/Prism.Core/Modularity/IModuleInfoGroup.cs
+++ b/src/Prism.Core/Modularity/IModuleInfoGroup.cs
@@ -4,10 +4,20 @@
 namespace Prism.Modularity
 {
     // IList must be supported in Silverlight 2 to be able to add items from XAML
+    /// <summary>
+    /// A collection of <see cref="ModuleInfo"/> for the Modules used by the application
+    /// </summary>
     public interface IModuleInfoGroup : IModuleCatalogItem, IList<IModuleInfo>, IList
     {
+        /// <summary>
+        /// When Prism should Initialize the module
+        /// <see cref="InitializationMode"/>
+        /// </summary>
         InitializationMode InitializationMode { get; set; }
 
+        /// <summary>
+        /// A string ref for the module for registration/resolution
+        /// </summary>
         string Ref { get; set; }
     }
 }
diff --git a/src/Wpf/Prism.DryIoc.Wpf/Legacy/DryIocExtensions.cs b/src/Wpf/Prism.DryIoc.Wpf/Legacy/DryIocExtensions.cs
index 8dd46c3533..8d3fdcf147 100644
--- a/src/Wpf/Prism.DryIoc.Wpf/Legacy/DryIocExtensions.cs
+++ b/src/Wpf/Prism.DryIoc.Wpf/Legacy/DryIocExtensions.cs
@@ -10,6 +10,7 @@ public static class DryIocExtensions
         /// Registers an object for navigation.
         /// </summary>
         /// <typeparam name="T">The Type of the object to register</typeparam>
+        /// <param name="container">The container instance</param>
         /// <param name="name">The unique name to register with the object</param>
         public static void RegisterTypeForNavigation<T>(this IContainer container, string name = null)
         {

From e4a525aca0fa05bb3f9f474d9d49517c3dc34fe8 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 13:23:12 -0400
Subject: [PATCH 02/16] Update ParametersBase.cs

Adding comments per request of Dan Siegel
---
 src/Prism.Core/Common/ParametersBase.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index d720aa870c..0343d80b8c 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -8,7 +8,7 @@
 namespace Prism.Common
 {
     /// <summary>
-    /// Base class for Navigation parameters
+    /// This is a generic parameters base class used for Dialog Parameters and Navigation Parameters
     /// </summary>
     public abstract class ParametersBase : IEnumerable<KeyValuePair<string, object>>
     {

From e0e889fa88be3bf9ae448fe21ef6af58f7921566 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 13:31:10 -0400
Subject: [PATCH 03/16] Update ParametersBase.cs

I have changed the file in an attempt to reduce the confusion identified by Dan Siegel
---
 src/Prism.Core/Common/ParametersBase.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index 0343d80b8c..55fcd962bf 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -102,7 +102,7 @@ public object this[string key]
             _entries.Select(x => x.Key);
 
         /// <summary>
-        /// Adds the KeyValuePair to the Collection
+        /// Adds the <paramref name="Key" /> and <paramref name="Value" /> to the KeyValuePair<string,object> collection
         /// </summary>
         /// <param name="key"></param>
         /// <param name="value"></param>

From 8f045c27ed6d30ea0f75fdcb02db3b49c9165b43 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 13:37:46 -0400
Subject: [PATCH 04/16] Update ParametersBase.cs

I have added the following comment to the key parameters
---
 src/Prism.Core/Common/ParametersBase.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index 55fcd962bf..ab2b1b2290 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -72,7 +72,7 @@ protected ParametersBase(string query)
         /// Searches Parameter collection and returns value if Collection contains key.
         /// Otherswise returns null.
         /// </summary>
-        /// <param name="key"></param>
+        /// <param name="key">This parameter represents the key that represents the value to be returned</param>
         /// <returns></returns>
         public object this[string key]
         {
@@ -102,7 +102,7 @@ public object this[string key]
             _entries.Select(x => x.Key);
 
         /// <summary>
-        /// Adds the <paramref name="Key" /> and <paramref name="Value" /> to the KeyValuePair<string,object> collection
+        /// Adds the <paramref name="key" /> and <paramref name="value" /> to the KeyValuePair<string,object> collection
         /// </summary>
         /// <param name="key"></param>
         /// <param name="value"></param>

From 0c9d30cc567a12d6dbdd3ba05df45853a8e37488 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 13:43:50 -0400
Subject: [PATCH 05/16] Update ParametersBase.cs

Added comment to the <param> tag for method documentation::
<param name="key">The key for the value to be returned</param>

Added following comment tot he <returns> tag for method documentation:
Returns a matching parameter of <typeparam name="T" /> if one exists in the Collection
---
 src/Prism.Core/Common/ParametersBase.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index ab2b1b2290..5a42728f32 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -72,7 +72,7 @@ protected ParametersBase(string query)
         /// Searches Parameter collection and returns value if Collection contains key.
         /// Otherswise returns null.
         /// </summary>
-        /// <param name="key">This parameter represents the key that represents the value to be returned</param>
+        /// <param name="key">The key for the value to be returned</param>
         /// <returns></returns>
         public object this[string key]
         {
@@ -127,8 +127,8 @@ public IEnumerator<KeyValuePair<string, object>> GetEnumerator() =>
         /// Returns the value of the member referenced by key
         /// </summary>
         /// <typeparam name="T">The type of object to be returned</typeparam>
-        /// <param name="key"></param>
-        /// <returns></returns>
+        /// <param name="key">The key for the value to be returned</param>
+        /// <returns>Returns a matching parameter of <typeparam name="T" /> if one exists in the Collection</returns>
         public T GetValue<T>(string key) => 
             _entries.GetValue<T>(key);
 

From b58f2752126fd5097a704457edc83c4b9c1e775b Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 13:47:16 -0400
Subject: [PATCH 06/16] Update ParametersBase.cs

Modified <param> tags on a few methods that check existence of keys/values or return values
---
 src/Prism.Core/Common/ParametersBase.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index 5a42728f32..cd1b037a58 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -104,15 +104,15 @@ public object this[string key]
         /// <summary>
         /// Adds the <paramref name="key" /> and <paramref name="value" /> to the KeyValuePair<string,object> collection
         /// </summary>
-        /// <param name="key"></param>
-        /// <param name="value"></param>
+        /// <param name="key">The key to reference this value in the KeyValuePair<string, object></param>
+        /// <param name="value">The value of the parameter to store</param>
         public void Add(string key, object value) =>
             _entries.Add(new KeyValuePair<string, object>(key, value));
 
         /// <summary>
         /// Checks collection for presense of key
         /// </summary>
-        /// <param name="key"></param>
+        /// <param name="key">The key to check in the collection</param>
         /// <returns>True if key exists; else returns false.</returns>
         public bool ContainsKey(string key) =>
             _entries.ContainsKey(key);

From cb32be06d96e22630bbc78643682be39ed49d9ba Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 14:45:04 -0400
Subject: [PATCH 07/16] Update IModuleInfo.cs

Fixed Indentation
---
 src/Prism.Core/Modularity/IModuleInfo.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index 3896b210e8..71c0c855c9 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -5,7 +5,7 @@ namespace Prism.Modularity
     /// <summary>
     /// Set of properties for each Module
     /// </summary>
-public interface IModuleInfo : IModuleCatalogItem
+    public interface IModuleInfo : IModuleCatalogItem
     {
         /// <summary>
         /// The modules this instance depends on.
@@ -34,4 +34,4 @@ public interface IModuleInfo : IModuleCatalogItem
         /// </summary>
         ModuleState State { get; set; }
     }
-}
\ No newline at end of file
+}

From d23846c01bf4985b45b5a2fbb8da1a3affb57591 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 14:48:04 -0400
Subject: [PATCH 08/16] Update IModuleInfo.cs

---
 src/Prism.Core/Modularity/IModuleInfo.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index 71c0c855c9..e2ec7f71fe 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -8,7 +8,7 @@ namespace Prism.Modularity
     public interface IModuleInfo : IModuleCatalogItem
     {
         /// <summary>
-        /// The modules this instance depends on.
+        /// The module names this instance depends on.
         /// </summary>
         Collection<string> DependsOn { get; set; }
         /// <summary>

From 534891b98a66a284cdcfa2cee1a0bc9104353bf8 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 14:54:01 -0400
Subject: [PATCH 09/16] Update IModuleInfo.cs

Updated comment for InitializationMode
---
 src/Prism.Core/Modularity/IModuleInfo.cs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index e2ec7f71fe..437ba4f38e 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -12,8 +12,7 @@ public interface IModuleInfo : IModuleCatalogItem
         /// </summary>
         Collection<string> DependsOn { get; set; }
         /// <summary>
-        /// When Prism should Initialize the module
-        /// <see cref="InitializationMode"/>
+        /// Gets or Sets the <see cref="InitializationMode" />
         /// </summary>
         InitializationMode InitializationMode { get; set; }
         /// <summary>

From 79c621169c82cbbef6df391df4231b60be6117bd Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 14:56:57 -0400
Subject: [PATCH 10/16] Update IModuleInfo.cs

Added <ref> clarification and <remarks> tag as requested by Dan Siegel
---
 src/Prism.Core/Modularity/IModuleInfo.cs | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index 437ba4f38e..062d0dcf5b 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -11,22 +11,30 @@ public interface IModuleInfo : IModuleCatalogItem
         /// The module names this instance depends on.
         /// </summary>
         Collection<string> DependsOn { get; set; }
+        
         /// <summary>
         /// Gets or Sets the <see cref="InitializationMode" />
         /// </summary>
         InitializationMode InitializationMode { get; set; }
+        
         /// <summary>
         /// The name of the module
         /// </summary>
         string ModuleName { get; set; }
+        
         /// <summary>
         /// The module's type
         /// </summary>
         string ModuleType { get; set; }
+        
         /// <summary>
-        /// A string ref for the module for registration/resolution
+        /// A string ref is a location reference to load the module as it may not be already loaded in the Appdomain in some cases may need to be downloaded.
         /// </summary>
+        /// <Remarks>
+        /// This is only used for WPF
+        /// </Remarks>
         string Ref { get; set; }
+
         /// <summary>
         /// An enum that lists the modules state
         /// <see cref="ModuleState"/>

From c5f9a387c5e345ad6f24eaca40fce166c5896404 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 14:57:54 -0400
Subject: [PATCH 11/16] Update IModuleInfo.cs

Update comment on ModuleState
---
 src/Prism.Core/Modularity/IModuleInfo.cs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfo.cs b/src/Prism.Core/Modularity/IModuleInfo.cs
index 062d0dcf5b..7ef81c2f27 100644
--- a/src/Prism.Core/Modularity/IModuleInfo.cs
+++ b/src/Prism.Core/Modularity/IModuleInfo.cs
@@ -36,8 +36,7 @@ public interface IModuleInfo : IModuleCatalogItem
         string Ref { get; set; }
 
         /// <summary>
-        /// An enum that lists the modules state
-        /// <see cref="ModuleState"/>
+        /// Gets or Sets the current <see cref="ModuleState" />
         /// </summary>
         ModuleState State { get; set; }
     }

From 1d817a7fbd8925c6230f9e1663e78e2896b92183 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Fri, 17 Apr 2020 15:00:41 -0400
Subject: [PATCH 12/16] Update IModuleInfoGroup.cs

Add ref summary info as reqeusted
---
 src/Prism.Core/Modularity/IModuleInfoGroup.cs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/Prism.Core/Modularity/IModuleInfoGroup.cs b/src/Prism.Core/Modularity/IModuleInfoGroup.cs
index 3e021d4ff1..c12ebf1fc0 100644
--- a/src/Prism.Core/Modularity/IModuleInfoGroup.cs
+++ b/src/Prism.Core/Modularity/IModuleInfoGroup.cs
@@ -16,8 +16,11 @@ public interface IModuleInfoGroup : IModuleCatalogItem, IList<IModuleInfo>, ILis
         InitializationMode InitializationMode { get; set; }
 
         /// <summary>
-        /// A string ref for the module for registration/resolution
+        /// A string ref is a location reference to load the module as it may not be already loaded in the Appdomain in some cases may need to be downloaded.
         /// </summary>
+        /// <Remarks>
+        /// This is only used for WPF
+        /// </Remarks>
         string Ref { get; set; }
     }
 }

From bc394334a254939d10c1565fc4f950d5a55aedb4 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Sat, 18 Apr 2020 23:10:26 -0400
Subject: [PATCH 13/16] Update ParametersBase.cs

I have added text to some of the tags as requested by Dan Siegel
---
 src/Prism.Core/Common/ParametersBase.cs | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index cd1b037a58..4df3eb0e2d 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -113,7 +113,7 @@ public void Add(string key, object value) =>
         /// Checks collection for presense of key
         /// </summary>
         /// <param name="key">The key to check in the collection</param>
-        /// <returns>True if key exists; else returns false.</returns>
+        /// <returns><c>true</c> if key exists; else returns <c>false</c>.</returns>
         public bool ContainsKey(string key) =>
             _entries.ContainsKey(key);
 
@@ -123,19 +123,20 @@ public bool ContainsKey(string key) =>
         /// <returns>Enumerator</returns>
         public IEnumerator<KeyValuePair<string, object>> GetEnumerator() =>
             _entries.GetEnumerator();
+        
         /// <summary>
         /// Returns the value of the member referenced by key
         /// </summary>
         /// <typeparam name="T">The type of object to be returned</typeparam>
         /// <param name="key">The key for the value to be returned</param>
-        /// <returns>Returns a matching parameter of <typeparam name="T" /> if one exists in the Collection</returns>
+        /// <returns>Returns a matching parameter of <see cref="T" /> if one exists in the Collection</returns>
         public T GetValue<T>(string key) => 
             _entries.GetValue<T>(key);
 
         /// <summary>
         /// Returns an IEnumerable of all parameters 
         /// </summary>
-        /// <typeparam name="T"></typeparam>
+        /// <typeparam name="T">The type for the values to be returned</typeparam>
         /// <param name="key"></param>
         public IEnumerable<T> GetValues<T>(string key) =>
             _entries.GetValues<T>(key);
@@ -143,7 +144,7 @@ public IEnumerable<T> GetValues<T>(string key) =>
         /// <summary>
         /// Checks to see if the parameter collection contains the value
         /// </summary>
-        /// <typeparam name="T"></typeparam>
+        /// <typeparam name="T">The type for the values to be returned</typeparam>
         /// <param name="key"></param>
         /// <param name="value">Value of the returned parameter if it exists</param>
         public bool TryGetValue<T>(string key, out T value) =>
@@ -155,7 +156,7 @@ IEnumerator IEnumerable.GetEnumerator() =>
         /// <summary>
         /// Converts parameter collection to a parameter string
         /// </summary>
-        /// <returns></returns>
+        /// <returns>A string representation of the parameters</returns>
         public override string ToString()
         {
             var queryBuilder = new StringBuilder();
@@ -188,7 +189,7 @@ public override string ToString()
         /// <summary>
         /// Adds a collection of parameters to the local parameter list
         /// </summary>
-        /// <param name="parameters"></param>
+        /// <param name="parameters">An IEnumberable of KeyValuePairs to add to the current parameter list</param>
         [EditorBrowsable(EditorBrowsableState.Never)]
         public void FromParameters(IEnumerable<KeyValuePair<string, object>> parameters) =>
             _entries.AddRange(parameters);

From aad61dd671618775639e5d29335c407853811396 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Mon, 20 Apr 2020 11:33:57 -0400
Subject: [PATCH 14/16] Update ParametersBase.cs

I have added text to all the empty tags I found
---
 src/Prism.Core/Common/ParametersBase.cs | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index 4df3eb0e2d..ff81e8fc7b 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -73,7 +73,7 @@ protected ParametersBase(string query)
         /// Otherswise returns null.
         /// </summary>
         /// <param name="key">The key for the value to be returned</param>
-        /// <returns></returns>
+        /// <returns>The value of the parameter referenced by the key; otherwise <c>null</c></returns>
         public object this[string key]
         {
             get
@@ -137,7 +137,7 @@ public T GetValue<T>(string key) =>
         /// Returns an IEnumerable of all parameters 
         /// </summary>
         /// <typeparam name="T">The type for the values to be returned</typeparam>
-        /// <param name="key"></param>
+        /// <param name="key">Returns a IEnumberable of all the instances of type <see cref="T"/></param>
         public IEnumerable<T> GetValues<T>(string key) =>
             _entries.GetValues<T>(key);
 
@@ -145,7 +145,7 @@ public IEnumerable<T> GetValues<T>(string key) =>
         /// Checks to see if the parameter collection contains the value
         /// </summary>
         /// <typeparam name="T">The type for the values to be returned</typeparam>
-        /// <param name="key"></param>
+        /// <param name="key">The key for the value to be returned</param>
         /// <param name="value">Value of the returned parameter if it exists</param>
         public bool TryGetValue<T>(string key, out T value) =>
             _entries.TryGetValue(key, out value);

From c9d0ef4c22cb7e736578bac32f5f336e2a7e7c4b Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Mon, 20 Apr 2020 11:35:57 -0400
Subject: [PATCH 15/16] Update ParametersBase.cs

Fixed a typo
---
 src/Prism.Core/Common/ParametersBase.cs | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index ff81e8fc7b..94d7949abd 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -137,7 +137,8 @@ public T GetValue<T>(string key) =>
         /// Returns an IEnumerable of all parameters 
         /// </summary>
         /// <typeparam name="T">The type for the values to be returned</typeparam>
-        /// <param name="key">Returns a IEnumberable of all the instances of type <see cref="T"/></param>
+        /// <param name="key">The key for the values to be returned</param>
+        ///<returns>Returns a IEnumberable of all the instances of type <see cref="T"/></returns>
         public IEnumerable<T> GetValues<T>(string key) =>
             _entries.GetValues<T>(key);
 

From f92701b2da28422e666a86489c57186365fbb858 Mon Sep 17 00:00:00 2001
From: d3fkn1ght <gdudics@gmail.com>
Date: Tue, 21 Apr 2020 09:21:30 -0400
Subject: [PATCH 16/16] Update ParametersBase.cs

Removed paramref
---
 src/Prism.Core/Common/ParametersBase.cs | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Prism.Core/Common/ParametersBase.cs b/src/Prism.Core/Common/ParametersBase.cs
index 94d7949abd..9d9ce57a23 100644
--- a/src/Prism.Core/Common/ParametersBase.cs
+++ b/src/Prism.Core/Common/ParametersBase.cs
@@ -102,7 +102,7 @@ public object this[string key]
             _entries.Select(x => x.Key);
 
         /// <summary>
-        /// Adds the <paramref name="key" /> and <paramref name="value" /> to the KeyValuePair<string,object> collection
+        /// Adds the key and value to the KeyValuePair<string,object> collection
         /// </summary>
         /// <param name="key">The key to reference this value in the KeyValuePair<string, object></param>
         /// <param name="value">The value of the parameter to store</param>