clients/organisation: navigate to last visited organisation

[magarpratik]

Closes #4619

This PR add functionality to remember last visited organization.

When an authenticated user visits the default home page, they will be redirected to the last visited organization's dashboard.

remember_last_visited_org.mov

[vercel]

@magarpratik is attempting to deploy a commit to the polar-sh Team on Vercel.

A member of the Team first needs to authorize it.

[frankie567]

Thank you for this @magarpratik 👍 That's interesting, but I would have liked a server-side approach. Basically, the logic is handled on the following route:

polar/clients/apps/web/src/app/(main)/start/page.tsx

Lines 14 to 23 in 07a4c0a

	export default async function Page() {
	const api = getServerSideAPI()
	const userOrganizations = await getUserOrganizations(api)
	if (userOrganizations.length === 0) {
	redirect('/purchases')
	}
	redirect(`/dashboard/${userOrganizations[0].slug}`)
	}

Ideally, this would check for the existence of a cookie (instead of local storage) to directly redirect to the last organization at that point.

This cookie could be then be set client-side on the DashboardLayout component for example:

polar/clients/apps/web/src/components/Layout/DashboardLayout.tsx

Lines 91 to 120 in ca27097

	const DashboardLayout = (
	props: PropsWithChildren<{
	breadcrumb?: React.ReactNode
	className?: string
	}>,
	) => {
	const { organization } = useContext(MaintainerOrganizationContext)
	return (
	<DashboardProvider organization={organization}>
	<div className="relative flex h-full w-full flex-col gap-x-8 bg-gray-100 md:flex-row md:p-6 dark:bg-transparent">
	<MobileNav />
	<div className="hidden md:flex">
	<DashboardSidebar />
	</div>
	<div
	className={twMerge(
	'relative flex h-full w-full flex-col gap-y-4',
	props.className,
	)}
	>
	<DashboardTopbar breadcrumb={props.breadcrumb} />
	{/* On large devices, scroll here. On small devices the _document_ is the only element that should scroll. */}
	<main className="relative flex min-h-0 w-full flex-grow flex-col">
	{props.children}
	</main>
	</div>
	</div>
	</DashboardProvider>
	)
	}

What do you think? :)

[magarpratik]

Hi @frankie567,

Thank you for the feedback. 🙌

It makes sense and it would probably be a bit more efficient too. I'll give it a go. 🫡

[magarpratik]

Hi @frankie567, I've implemented your feedback.

PR is ready for re-review. 👍

[frankie567]

Looks great! Just need to bump that cookie expiration and we'll be good to go!

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
polar	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Dec 18, 2024 8:46am
polar-sandbox	🛑 Canceled (Inspect)			Dec 18, 2024 8:46am