From 77df15eeaacd2aab17df60eb67f7c378a0deae48 Mon Sep 17 00:00:00 2001
From: Ronald Barendse <ronald@panoramastudios.nl>
Date: Fri, 4 Oct 2019 00:46:26 +0200
Subject: [PATCH] Added remarks about property serialization

---
 src/Umbraco.Core/Exceptions/DataOperationException.cs   | 3 +++
 src/Umbraco.Core/Persistence/EntityNotFoundException.cs | 3 +++
 src/Umbraco.Web/Install/InstallException.cs             | 3 +++
 3 files changed, 9 insertions(+)

diff --git a/src/Umbraco.Core/Exceptions/DataOperationException.cs b/src/Umbraco.Core/Exceptions/DataOperationException.cs
index 4248377e3d..904420e1cf 100644
--- a/src/Umbraco.Core/Exceptions/DataOperationException.cs
+++ b/src/Umbraco.Core/Exceptions/DataOperationException.cs
@@ -17,6 +17,9 @@ namespace Umbraco.Core.Exceptions
         /// <value>
         /// The operation.
         /// </value>
+        /// <remarks>
+        /// This object should be serializable to prevent a <see cref="SerializationException" /> to be thrown.
+        /// </remarks>
         public T Operation { get; private set; }
 
         /// <summary>
diff --git a/src/Umbraco.Core/Persistence/EntityNotFoundException.cs b/src/Umbraco.Core/Persistence/EntityNotFoundException.cs
index 6bcf6a269d..b8778f0975 100644
--- a/src/Umbraco.Core/Persistence/EntityNotFoundException.cs
+++ b/src/Umbraco.Core/Persistence/EntityNotFoundException.cs
@@ -18,6 +18,9 @@ namespace Umbraco.Core.Persistence
         /// <value>
         /// The identifier.
         /// </value>
+        /// <remarks>
+        /// This object should be serializable to prevent a <see cref="SerializationException" /> to be thrown.
+        /// </remarks>
         public object Id { get; private set; }
 
         /// <summary>
diff --git a/src/Umbraco.Web/Install/InstallException.cs b/src/Umbraco.Web/Install/InstallException.cs
index 44f6e4fe50..7efcbdc769 100644
--- a/src/Umbraco.Web/Install/InstallException.cs
+++ b/src/Umbraco.Web/Install/InstallException.cs
@@ -24,6 +24,9 @@ namespace Umbraco.Web.Install
         /// <value>
         /// The view model.
         /// </value>
+        /// <remarks>
+        /// This object should be serializable to prevent a <see cref="SerializationException" /> to be thrown.
+        /// </remarks>
         public object ViewModel { get; private set; }
 
         /// <summary>