Merge pull request #717 from bart-degreed/resource-hooks-disabled

Disabled resource hooks by default, as they are still experimental

Author: Bart Koelman

src/Examples/JsonApiDotNetCoreExample/Startups/Startup.cs

@@ -45,6 +45,7 @@ public virtual void ConfigureServices(IServiceCollection services)
                     options.IncludeTotalRecordCount = true;
                     options.LoadDatabaseValues = true;
                     options.ValidateModelState = true;
+                    options.EnableResourceHooks = true;
                 },
                 discovery => discovery.AddCurrentAssembly());
             // once all tests have been moved to WebApplicationFactory format we can get rid of this line below

src/JsonApiDotNetCore/Configuration/JsonApiOptions.cs

@@ -30,11 +30,10 @@ public class JsonApiOptions : IJsonApiOptions
         public bool IncludeExceptionStackTraceInErrors { get; set; } = false;
 
         /// <summary>
-        /// Whether or not ResourceHooks are enabled. 
-        /// 
-        /// Default is set to <see langword="true"/>
+        /// Whether or not resource hooks are enabled. 
+        /// This is currently an experimental feature and defaults to <see langword="false"/>.
         /// </summary>
-        public bool EnableResourceHooks { get; set; } = true;
+        public bool EnableResourceHooks { get; set; } = false;
 
         /// <summary>
         /// Whether or not database values should be included by default