Skip to content

chore: Sentry cleanup#4314

Merged
dalkia merged 5 commits into
devfrom
chore/sentry-mantainance
Jun 6, 2025
Merged

chore: Sentry cleanup#4314
dalkia merged 5 commits into
devfrom
chore/sentry-mantainance

Conversation

@dalkia

@dalkia dalkia commented May 30, 2025

Copy link
Copy Markdown
Collaborator

Pull Request Description

What does this PR change?

  • Adding breadcrumb for UnityWebRequestException in the hope to shed light to this kind of sentry exception
  • Adds breadcrumbs to all supressed exception in the hope to add more info for Snetry debugging
  • Some cleanup of AddSentryScopeConfigurator. No need to have an external startup operation, both the scene and realm lambda operation can be added at the same moment for some centralization

Test Instructions

Test Steps

  1. Just a smoke test

Code Review Reference

Please review our Code Review Standards before submitting.

@github-actions

github-actions Bot commented May 30, 2025

Copy link
Copy Markdown
Contributor

@dalkia dalkia self-assigned this May 30, 2025
@dalkia
dalkia marked this pull request as ready for review June 5, 2025 17:26
@dalkia
dalkia requested review from a team as code owners June 5, 2025 17:26

@Ludmilafantaniella Ludmilafantaniella left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Regressions for this ticket had been performed in order to verify that the normal flow is working as expected:

  • ✔️ Changing accounts
  • ✔️ Backpack and wearables in world
  • ✔️ Emotes in world and in backpack
  • ✔️ Teleport with map/coordinates/Jump In
  • ✔️ Chat and multiplayer
  • ✔️ Profile card
  • ✔️ Camera
  • ✔️ Skybox

@dalkia
dalkia enabled auto-merge (squash) June 6, 2025 13:25
@dalkia
dalkia merged commit 2491461 into dev Jun 6, 2025
@dalkia
dalkia deleted the chore/sentry-mantainance branch June 6, 2025 14:01
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.

3 participants