Skip to content

Middleware refactoring - #166

Merged
filopedraz merged 6 commits into
mainfrom
feat/middleware-refactor
Oct 13, 2025
Merged

Middleware refactoring#166
filopedraz merged 6 commits into
mainfrom
feat/middleware-refactor

Conversation

@panos-the-man

@panos-the-man panos-the-man commented Oct 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • Simplify useEffect in use-active-organization hook
  • Simplify middleware, keep it light (making requests there could eventually slow down the app)
  • The redirect logic relies on publicMetadata?.onboardingComplete. In Clerk we need to include the publicMetadata in the session. See:
Screenshot 2025-10-13 at 09 24 56

@panos-the-man
panos-the-man marked this pull request as ready for review October 13, 2025 07:35
@filopedraz
filopedraz merged commit dff6b71 into main Oct 13, 2025
2 checks passed
@filopedraz
filopedraz deleted the feat/middleware-refactor branch October 13, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@filopedraz filopedraz Awaiting requested review from filopedraz

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@panos-the-man @filopedraz