hotfix: session bugging a bit when clicking around

[chitalian]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
helicone	❌ Failed (Inspect)			Mar 18, 2025 10:34pm
helicone-bifrost	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 18, 2025 10:34pm
helicone-eu	❌ Failed (Inspect)			Mar 18, 2025 10:34pm

[promptless]

✅ No documentation updates required.

[greptile-apps]

PR Summary

This PR addresses session page performance and loading state management issues, focusing on optimizing refresh behavior and consolidating loading states.

• Changed initial isLive state to false in /web/pages/sessions/[session_id].tsx to prevent automatic session refreshes
• Added consolidated isAnyLoading state in /web/components/templates/sessions/sessionsPage.tsx to streamline loading indicators
• Bug identified: Duplicate isLoading check in loading state conditions needs to be removed
• Wrapped SessionContent in React Fragment for proper component rendering

_{2 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings | Greptile}

[greptile-apps]

logic: isLoading is listed twice in the conditions, which is redundant and may mask other loading states

Suggested change

	return (
	isLoading ||
	allNames.isLoading ||
	allNames.isRefetching ||
	isLoading ||
	names.isLoading ||
	names.isRefetching
	);
	return (
	isLoading ||
	allNames.isLoading ||
	allNames.isRefetching ||
	names.isLoading ||
	names.isRefetching
	);

[fumedev]

Summary

• Introduces a new isAnyLoading state to handle multiple loading conditions in SessionsPage.
• Replaces direct loading checks with the new isAnyLoading state in the rendering logic.
• Changes the initial state of isLive in SessionDetail from true to false.
• Ensures SessionContent component is wrapped in a React fragment in SessionDetail.