chore: Updated sample configure callback (#1341)

getsentry · May 30, 2023 · 0e2cef5 · 0e2cef5
1 parent bf5e705
commit 0e2cef5
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Fixes
+
+- Updated sample configure callback to use the new `BeforeSend`methods that allow the use of `Hints` ([#1341](https://github.com/getsentry/sentry-unity/pull/1341))
+
 ### Dependencies
 
 - Bump Cocoa SDK from v8.7.2 to v8.7.3 ([#1342](https://github.com/getsentry/sentry-unity/pull/1342))

diff --git a/samples/unity-of-bugs/Assets/Scripts/BuildTimeConfiguration.cs b/samples/unity-of-bugs/Assets/Scripts/BuildTimeConfiguration.cs
@@ -2,7 +2,7 @@
 using Sentry.Unity;
 
 [CreateAssetMenu(fileName = "Assets/Resources/Sentry/BuildTimeConfiguration.asset", menuName = "Sentry/BuildTimeConfiguration", order = 999)]
-public class BuildTimeConfiguration : Sentry.Unity.SentryBuildTimeOptionsConfiguration
+public class BuildTimeConfiguration : SentryBuildTimeOptionsConfiguration
 {
     /// Called during app build. Changes made here will affect build-time processing, symbol upload, etc.
     /// Additionally, because iOS, macOS and Android native error handling is configured at build time,

diff --git a/samples/unity-of-bugs/Assets/Scripts/RuntimeConfiguration.cs b/samples/unity-of-bugs/Assets/Scripts/RuntimeConfiguration.cs
@@ -1,9 +1,10 @@
+using Sentry;
 using UnityEngine;
 using Sentry.Unity;
 using UnityEngine.SceneManagement;
 
 [CreateAssetMenu(fileName = "Assets/Resources/Sentry/RuntimeConfiguration.asset", menuName = "Sentry/RuntimeConfiguration", order = 999)]
-public class RuntimeConfiguration : Sentry.Unity.SentryRuntimeOptionsConfiguration
+public class RuntimeConfiguration : SentryRuntimeOptionsConfiguration
 {
     /// Called at the player startup by SentryInitialization.
     /// You can alter configuration for the C# error handling and also
@@ -25,7 +26,7 @@ public override void Configure(SentryUnityOptions options)
         }
 
         // BeforeSend is only relevant at runtime. It wouldn't hurt to be set at build time, just wouldn't do anything.
-        options.BeforeSend = sentryEvent =>
+        options.SetBeforeSend((sentryEvent, _) =>
         {
             if (sentryEvent.Tags.ContainsKey("SomeTag"))
             {
@@ -34,7 +35,7 @@ public override void Configure(SentryUnityOptions options)
             }
 
             return sentryEvent;
-        };
+        });
 
         Debug.Log(nameof(RuntimeConfiguration) + "::Configure() finished");
     }