Skip to content

feat: Unity Asset Store Integration #2131

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bitsandfoxes
Copy link
Contributor

Releasing to the Unity Asset Store requires manual import of the package and upload through the Editor. This means we do have access to the actual source that is getting shipped and enables us to add

options.AddIntegration(new SentryUnityAssetStoreIntegration());

to the SentryIntegrations

public static class SentryIntegrations
{
public static void Configure(SentryUnityOptions options)

that is getting shipped with the package and compiled with the game.

@bitsandfoxes bitsandfoxes changed the title added asset store integration feat: Unity Asset Store Integration Apr 28, 2025
Copy link
Contributor

Fails
🚫 Please consider adding a changelog entry for the next release.
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Unity Asset Store Integration ([#2131](https://github.com/getsentry/sentry-unity/pull/2131))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against 5e888c7

{
if (options is SentryUnityOptions unityOptions)
{
unityOptions.SdkIntegrationNames.Add("IL2CPPLineNumbers");
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
unityOptions.SdkIntegrationNames.Add("IL2CPPLineNumbers");
unityOptions.SdkIntegrationNames.Add("AssetStore");

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant