mirror of
https://github.com/windmill-labs/windmill.git
synced 2026-09-07 08:02:40 +00:00
fix(git-sync): fork PRs are opt-in on new connections too
Only auto-pull and fork sync default on for new app-backed connections; opening pull requests stays a deliberate per-repo decision. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PP5gBSPfo1YtkL1sWVAjJm
This commit is contained in:
co-authored by
Claude Fable 5
parent
8f08d0e460
commit
41f33b65c0
@@ -149,9 +149,9 @@
|
||||
isGithubApp = value?.is_github_app === true
|
||||
// A newly added connection defaults to pulling from Git only when
|
||||
// the repository is app-backed (instant webhook delivery). Polling
|
||||
// is opt-in for token repositories. Fork/dev workspaces never get
|
||||
// the parent-only defaults (the backend rejects them), and both
|
||||
// features are EE-only.
|
||||
// is opt-in for token repositories, PR opening is a deliberate
|
||||
// opt-in everywhere, and fork/dev workspaces never get the
|
||||
// parent-only defaults (the backend rejects them). EE-only.
|
||||
if (
|
||||
repo.isUnsavedConnection &&
|
||||
isGithubApp &&
|
||||
@@ -160,7 +160,6 @@
|
||||
repo.auto_pull === undefined
|
||||
) {
|
||||
repo.auto_pull = { enabled: true, mode: 'auto', sync_forks: true }
|
||||
repo.fork_open_prs = repo.fork_open_prs ?? true
|
||||
}
|
||||
// Webhook with polling fallback is the only delivery for app repos.
|
||||
if (isGithubApp && repo.auto_pull?.mode === 'polling') {
|
||||
|
||||
Reference in New Issue
Block a user