-
-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: More explicit Build Time Options Configuration
#1187
Conversation
src/Sentry.Unity.Editor/ConfigurationWindow/OptionsConfigurationTab.cs
Outdated
Show resolved
Hide resolved
…ry-unity into fix/rename-config
Build Time Options Configuration
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
collection of nits and notes after a quick pass half the way through
type: 2} | ||
<BuildtimeOptionsConfiguration>k__BackingField: {fileID: 11400000, guid: 6bcc81a646c08ce439ab03805d477458, | ||
<BuildTimeOptionsConfiguration>k__BackingField: {fileID: 11400000, guid: 1c61999bd2a61408393cfe218d42287e, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this a breaking change? If a user upgrades the SDK do they lose their option?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Unfortunately. I aim to include this as the 1.0.0
release to make the breaking change more palatable.
samples/unity-of-bugs/Assets/Scripts/BuildtimeOptionsConfiguration.cs
Outdated
Show resolved
Hide resolved
samples/unity-of-bugs/Assets/Scripts/BuildtimeOptionsConfiguration.cs
Outdated
Show resolved
Hide resolved
samples/unity-of-bugs/Assets/Scripts/BuildtimeOptionsConfiguration.cs
Outdated
Show resolved
Hide resolved
src/Sentry.Unity.Editor/ConfigurationWindow/OptionsConfigurationTab.cs
Outdated
Show resolved
Hide resolved
CHANGELOG.md
Outdated
@@ -8,6 +8,8 @@ | |||
- The option is still available from within the `SentryBuildTimeOptionsConfiguration` | |||
- Renamed `OptionsConfiguration` to `RuntimeOptionsConfiguration` on the ScriptableSentryOptions ([#1196](https://github.com/getsentry/sentry-unity/pull/1196)) | |||
- If you make use of programmatic runtime options configuration, you will need to reassign the scriptable object in the configuration tab | |||
- Renamed `SentryBuiltimeOptionsConfiguration` to `SentryBuilTimeOptionsConfiguration` ([#1187](https://github.com/getsentry/sentry-unity/pull/1187)) | |||
- If you make use of the programmatic build time configuration, you will need to update your implementation with the base class |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- If you make use of the programmatic build time configuration, you will need to update your implementation with the base class | |
### Note | |
- If you make use of the programmatic build time configuration, you will need to update your implementation with the base class |
Trying to be more upfront and visible about the difference between runtime and build time options.
Trying to emphasis
Build Time
vsRuntime
.