Skip to content

feat(seer settings): Frontend defaults for new auto_open_pr org level default#104207

Merged
Mihir-Mavalankar merged 6 commits intomasterfrom
fe-new-auto-pr-org-level-default
Dec 2, 2025
Merged

feat(seer settings): Frontend defaults for new auto_open_pr org level default#104207
Mihir-Mavalankar merged 6 commits intomasterfrom
fe-new-auto-pr-org-level-default

Conversation

@Mihir-Mavalankar
Copy link
Copy Markdown
Contributor

PR Details

@Mihir-Mavalankar Mihir-Mavalankar self-assigned this Dec 1, 2025
@github-actions github-actions bot added Scope: Frontend Automatically applied to PRs that change frontend components Scope: Backend Automatically applied to PRs that change backend components labels Dec 1, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 1, 2025

🚨 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.

Copy link
Copy Markdown
Member

@ryan953 ryan953 left a comment

Choose a reason for hiding this comment

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

backend stuff is all from #104205. i think if that merged first and this was rebased it would remove those files from this PR 👍

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 1, 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   #104207      +/-   ##
===========================================
- Coverage   80.61%    80.58%   -0.03%     
===========================================
  Files        9332      9322      -10     
  Lines      398491    398715     +224     
  Branches    25486     25336     -150     
===========================================
+ Hits       321227    321320      +93     
- Misses      76814     76943     +129     
- Partials      450       452       +2     

Keep only backend removal of set_default_project_auto_open_prs function.
Frontend implementation will be handled separately.
When seer-settings-gtm feature flag is enabled, use the org-level
autoOpenPrs setting to determine the default automatedRunStoppingPoint:
- autoOpenPrs=true -> default to 'open_pr'
- autoOpenPrs=false -> default to 'code_changes'

When feature flag is disabled, keep existing behavior (default to 'root_cause').
@Mihir-Mavalankar Mihir-Mavalankar force-pushed the fe-new-auto-pr-org-level-default branch from ffea3e7 to 945bf3b Compare December 1, 2025 23:01
@Mihir-Mavalankar Mihir-Mavalankar marked this pull request as ready for review December 1, 2025 23:02
@Mihir-Mavalankar
Copy link
Copy Markdown
Contributor Author

Backend PR has been merged and rebased this one with master.

@Mihir-Mavalankar Mihir-Mavalankar merged commit 0b6c64f into master Dec 2, 2025
48 checks passed
@Mihir-Mavalankar Mihir-Mavalankar deleted the fe-new-auto-pr-org-level-default branch December 2, 2025 18:35
@github-actions github-actions bot locked and limited conversation to collaborators Dec 18, 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.

2 participants