Skip to content

Conversation

@bfium
Copy link

@bfium bfium commented Dec 5, 2025

  • Add depends_on to frontend service to wait for backend healthcheck
  • Add beforeLoad guard to dashboard route to redirect to login when not authenticated
  • Prevents nginx 'host not found' errors and infinite loading spinner

…sers

- Add depends_on to frontend service to wait for backend healthcheck
- Add beforeLoad guard to dashboard route to redirect to login when not authenticated
- Prevents nginx 'host not found' errors and infinite loading spinner
@alejsdev alejsdev changed the title fix: ensure frontend waits for backend and redirect unauthenticated u… fix: ensure frontend waits for backend and redirect unauthenticated users Dec 5, 2025
@alejsdev alejsdev changed the title fix: ensure frontend waits for backend and redirect unauthenticated users 🐛 Ensure frontend waits for backend and redirect unauthenticated users Dec 5, 2025
@alejsdev alejsdev added the bug Something isn't working label Dec 5, 2025
@github-actions github-actions bot added the conflicts Automatically generated when a PR has a merge conflict label Dec 7, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 7, 2025

This pull request has a merge conflict that needs to be resolved.


export const Route = createFileRoute("/_layout/")({
component: Dashboard,
beforeLoad: async () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the parent layout's guard (_layout.tsx) executes before any child route's guard so this one here seems unnecessary 🤔

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh, just check, you are right👌🏾

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working conflicts Automatically generated when a PR has a merge conflict

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants