Skip to content

Conversation

constantinius
Copy link
Contributor

@constantinius constantinius commented Jul 10, 2025

  • Transform event classes to use @analytics.eventclass decorator
  • Transform analytics.record calls to use event class instances
  • Update imports as needed

Closes TET-837

Requires https://github.com/getsentry/getsentry/pull/18275

- Transform event classes to use @analytics.eventclass decorator
- Transform analytics.record calls to use event class instances
- Update imports as needed

Closes TET-837
@constantinius constantinius requested review from a team as code owners July 10, 2025 09:17
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 10, 2025
cursor[bot]

This comment was marked as outdated.

Copy link

codecov bot commented Jul 10, 2025

Codecov Report

❌ Patch coverage is 82.85714% with 6 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/sentry/receivers/experiments.py 57.14% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #95216      +/-   ##
==========================================
- Coverage   80.97%   80.63%   -0.34%     
==========================================
  Files        8531     8415     -116     
  Lines      376699   375156    -1543     
  Branches    23889    23634     -255     
==========================================
- Hits       305032   302515    -2517     
- Misses      71300    72283     +983     
+ Partials      367      358       -9     

Copy link
Member

@markstory markstory left a comment

Choose a reason for hiding this comment

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

Relocation event changes look good to me.

cursor[bot]

This comment was marked as outdated.

cursor[bot]

This comment was marked as outdated.

@getsantry getsantry bot added the Stale label Aug 8, 2025
@getsantry
Copy link
Contributor

getsantry bot commented Aug 8, 2025

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you remove the label Waiting for: Community, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@getsantry getsantry bot removed the Stale label Aug 9, 2025
cursor[bot]

This comment was marked as outdated.

@constantinius constantinius merged commit d45b25d into master Sep 4, 2025
64 checks passed
@constantinius constantinius deleted the constantinius/ref/analytics/tet-837 branch September 4, 2025 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants