Skip to content

combine multiple PRs#21

Open
arielweinberger wants to merge 8 commits intomainfrom
combine-steven-prs
Open

combine multiple PRs#21
arielweinberger wants to merge 8 commits intomainfrom
combine-steven-prs

Conversation

@arielweinberger
Copy link
Copy Markdown
Member

@arielweinberger arielweinberger commented Dec 30, 2025

So we can produce a final sandbox template image and run some tests.

arielweinberger and others added 8 commits December 30, 2025 17:37
* chore: update dependencies and lockfile versions

- Updated various Radix UI components to their latest versions.
- Upgraded several packages including React, TypeScript, and ESLint plugins.
- Adjusted package versions in package.json and bun.lock for consistency.

* bump appwrite to 21.1.0

---------

Co-authored-by: Ariel Weinberger <weinberger.ariel@gmail.com>
* feat: support client-side appwrite session

Expose Appwrite config client-side by prefixing env vars with VITE_ so they can be used in the client build.

Set Appwrite session cookie name to the standard Appwrite session cookie name so that if the cookie goes to Appwrite, it will be recognized.

* feat: update sign-out

Ensure the session is deleted from Appwrite.

* fix: set session cookie expiration to session expire

* fix: linting

---------

Co-authored-by: Steven Nguyen <stnguyen90@users.noreply.github.com>
Co-authored-by: Ariel Weinberger <weinberger.ariel@gmail.com>
* fix: remove unused query client and related exports

We should always use the QueryClient from the context rather than a separate QueryClient because using multiple QueryClients means data will be cached in different places and may require unnecessary fetches.

* chore: refactor router

- Remove getContext() to discourage using that to get the QueryClient
- Remove the custom TanstackQuery provider since setupRouterSsrQueryIntegration will automatically wrap the query client.

---------

Co-authored-by: Steven Nguyen <stnguyen90@users.noreply.github.com>
Co-authored-by: Ariel Weinberger <weinberger.ariel@gmail.com>
* chore: reuse currentUser from root loader

* chore: delete unused sign-out route

* fix: update getCurrentUser() to return null if session is invalid

If the session is invalid/expired, account.get() will throw an exception. This update catches the exception and returns null instead.

* chore: remove redirect from auth server functions

They are already handled client-side so we don't need them server-side.

In addition, doing it client-side allows us to invalidate whatever caches we need before redirecting to the new page.

* refactor: use TanStack Query for auth

Use TanStack Query so that API calls are cached, reducing unnecessary network requests and improving performance.

---------

Co-authored-by: Steven Nguyen <stnguyen90@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants