Fix Templates Banner on home page

[rSnapkoOpenOps]

Fixes OPS-3843

The banner is not visible in the templates dialog so I copied the check from there

openops/packages/react-ui/src/app/features/templates/components/select-flow-template-dialog-content.tsx

Lines 99 to 106 in 9b4caf0

	const { isConnectedToCloudTemplates } = useCloudProfile();
	const { createPollingInterval } = useUserInfoPolling();
	const useCloudTemplates = flagsHooks.useShouldFetchCloudTemplates();
	const { checkAccess } = useAuthorization();
	const hasWriteFlowPermission = checkAccess(Permission.WRITE_FLOW);
	const isFullCatalog =
	!isTemplatePreselected &&
	(isConnectedToCloudTemplates || !useCloudTemplates);

[linear]

OPS-3843 Templates banner shows for SSO users in Production

[Copilot]

Pull request overview

Updates the home page onboarding experience so the “Connect templates” banner and template filters use the same “cloud templates connected + flags” logic as the templates dialog, fixing cases where the banner was incorrectly hidden.

Changes:

• Replace cloudUser-based UI conditions with a derived isCloudUser based on useCloudProfile() and flagsHooks.useShouldFetchCloudTemplates().
• Update the templates carousel filter visibility and onboarding panels to use isCloudUser.
• Adjust the templates refetch/reset effect to trigger based on the new isCloudUser signal.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

isCloudUser depends on isConnectedToCloudTemplates from useCloudProfile(). In use-cloud-profile.ts, isConnectedToCloudTemplates is computed as data !== null, which evaluates to true while the React Query data is still undefined (initial/pending or when the query is disabled). That can incorrectly treat users as connected (hiding the connect banner / enabling filters) until the query resolves. Consider fixing useCloudProfile to treat undefined as not-connected (e.g., data != null plus isSuccess), or guard here by deriving connection from a value that cannot be undefined.

[rSnapkoOpenOps]

This is the same logic as used in the templates dialog, so it's fine

[alexandrudanpop]

this logic is really hard to follow, if it's common we should extract it and improve the code so the logic behind is more readable

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[rSnapkoOpenOps]

I extracted the common logic to reusable hook