Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

feat: Unity Asset Store Integration - #2131

#2131
Draft
bitsandfoxes wants to merge 1 commit into
maingetsentry/sentry-unity:mainfrom
feat/asset-store-integrationgetsentry/sentry-unity:feat/asset-store-integrationCopy head branch name to clipboard
Draft

feat: Unity Asset Store Integration#2131
bitsandfoxes wants to merge 1 commit into
maingetsentry/sentry-unity:mainfrom
feat/asset-store-integrationgetsentry/sentry-unity:feat/asset-store-integrationCopy head branch name to clipboard

Conversation

@bitsandfoxes

Copy link
Copy Markdown
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
@github-actions

Copy link
Copy Markdown
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
Copy Markdown
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.