Skip to content

feat(seer onboarding): add Seer onboarding endpoint#104818

Merged
srest2021 merged 14 commits intomasterfrom
srest2021/CW-48
Dec 12, 2025
Merged

feat(seer onboarding): add Seer onboarding endpoint#104818
srest2021 merged 14 commits intomasterfrom
srest2021/CW-48

Conversation

@srest2021
Copy link
Copy Markdown
Member

@srest2021 srest2021 commented Dec 11, 2025

fixes CW-48

Add an endpoint OrganizationSeerOnboardingEndpoint to handle Seer onboarding settings (to be used in the wizard). This endpoint calls the onboarding_seer_settings_update function.

Currently only supports RCA settings and project repo mappings. But per the ticket we could always add code review settings and the endpoint is named/designed generally.

@linear
Copy link
Copy Markdown

linear bot commented Dec 11, 2025

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Dec 11, 2025
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Dec 11, 2025
@github-actions
Copy link
Copy Markdown
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

@srest2021 srest2021 changed the title add onboarding endpoint feat(seer onboarding): add Seer onboarding endpoint Dec 11, 2025
@srest2021 srest2021 marked this pull request as ready for review December 11, 2025 22:43
@srest2021 srest2021 requested review from a team as code owners December 11, 2025 22:43
@srest2021 srest2021 requested review from a team and billyvg December 11, 2025 22:44
]

try:
onboarding_seer_settings_update(
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.

Function call and input formats look good.

Copy link
Copy Markdown
Member

@JoshFerge JoshFerge left a comment

Choose a reason for hiding this comment

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

API changes look good! will let others review + approve on the business logic! thank you!

Copy link
Copy Markdown
Member

@billyvg billyvg left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 12, 2025

Codecov Report

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

Files with missing lines Patch % Lines
...try/seer/endpoints/organization_seer_onboarding.py 97.46% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master   #104818      +/-   ##
===========================================
+ Coverage   80.41%    80.42%   +0.01%     
===========================================
  Files        9386      9386              
  Lines      403019    403002      -17     
  Branches    25929     25908      -21     
===========================================
+ Hits       324075    324134      +59     
+ Misses      78496     78420      -76     
  Partials      448       448              

@srest2021 srest2021 merged commit 62e8399 into master Dec 12, 2025
67 checks passed
@srest2021 srest2021 deleted the srest2021/CW-48 branch December 12, 2025 19:10
@github-actions github-actions bot locked and limited conversation to collaborators Dec 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants