Skip to content

fix(uptime): Don't consume a seat or create audit log when onboarding an auto detected monitor #92051

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

Merged
merged 1 commit into from
May 28, 2025

Conversation

wedamija
Copy link
Member

Users don't see onboarding monitors, and so they shouldn't consume a quota seat or create an audit log. This pr moves this to the point where the monitor graduates from onboarding.

@wedamija wedamija requested a review from a team as a code owner May 21, 2025 19:03
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 21, 2025
Copy link

codecov bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #92051   +/-   ##
=======================================
  Coverage   87.89%   87.90%           
=======================================
  Files       10226    10226           
  Lines      585974   585986   +12     
  Branches    22760    22760           
=======================================
+ Hits       515068   515084   +16     
+ Misses      70477    70473    -4     
  Partials      429      429           

@wedamija wedamija force-pushed the danf/uptime-disable-autodetection-on-manual-creation branch from 92b3be9 to b96c17d Compare May 27, 2025 17:13
Base automatically changed from danf/uptime-disable-autodetection-on-manual-creation to master May 28, 2025 20:53
… an auto detected monitor

Users don't see onboarding monitors, and so they shouldn't consume a quota seat or create an audit log. This pr moves this to the point where the monitor graduates from onboarding.
@wedamija wedamija force-pushed the danf/uptime-onboarding-dont-consume-seat-audit branch from 309d31c to 6bf5126 Compare May 28, 2025 20:55
@wedamija wedamija enabled auto-merge (squash) May 28, 2025 20:55
@wedamija wedamija merged commit e0c28ee into master May 28, 2025
60 checks passed
@wedamija wedamija deleted the danf/uptime-onboarding-dont-consume-seat-audit branch May 28, 2025 21:14
andrewshie-sentry pushed a commit that referenced this pull request Jun 2, 2025
… an auto detected monitor (#92051)

Users don't see onboarding monitors, and so they shouldn't consume a
quota seat or create an audit log. This pr moves this to the point where
the monitor graduates from onboarding.
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