fixes isMaybeConnected to let it return undefined when the value is unknown

sergeibbb · sergeibbb · commit 5b7b9ce2a549 · 2025-03-12T15:20:37.000+01:00
diff --git a/src/plus/integrations/integrationService.ts b/src/plus/integrations/integrationService.ts
@@ -1010,9 +1010,9 @@ export class IntegrationService implements Disposable {
 		return this.getMyPullRequestsCore(integrations);
 	}
 
-	isMaybeConnected(remote: GitRemote): boolean {
+	isMaybeConnected(remote: GitRemote): boolean | undefined {
 		if (remote.provider?.id != null && this.supports(remote.provider.id)) {
-			return this.getByRemoteCached(remote)?.maybeConnected ?? false;
+			return this.getByRemoteCached(remote)?.maybeConnected;
 		}
 		return false;
 	}

Original file line number	Diff line number	Diff line change
`@@ -1010,9 +1010,9 @@ export class IntegrationService implements Disposable {`
`1010`	`1010`	`return this.getMyPullRequestsCore(integrations);`
`1011`	`1011`	`}`
`1012`	`1012`
`1013`		`- isMaybeConnected(remote: GitRemote): boolean {`
	`1013`	`+ isMaybeConnected(remote: GitRemote): boolean \| undefined {`
`1014`	`1014`	`if (remote.provider?.id != null && this.supports(remote.provider.id)) {`
`1015`		`- return this.getByRemoteCached(remote)?.maybeConnected ?? false;`
	`1015`	`+ return this.getByRemoteCached(remote)?.maybeConnected;`
`1016`	`1016`	`}`
`1017`	`1017`	`return false;`
`1018`	`1018`	`}`