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👌🏾

@tiangolo
Copy link
Member

I'll pass on this for now. ☕

@tiangolo tiangolo closed this Jan 22, 2026
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.

3 participants