chore: add logout to create org modal#6103
chore: add logout to create org modal#6103sheensantoscapadngan wants to merge 3 commits intomainfrom
Conversation
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7f90b7a2ff
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| <button | ||
| type="button" | ||
| className="text-sm text-mineshaft-400 underline-offset-4 hover:text-mineshaft-200 hover:underline" | ||
| onClick={() => logout.mutate()} |
There was a problem hiding this comment.
Redirect user to login after logout action
Clicking the new "Log out" control only calls logout.mutate() and never navigates away, so in the NoOrgPage flow (CreateOrgModal rendered without onClose) the user can remain on /_authenticate/organizations/none even after session state is cleared. Because auth checks for this route run in beforeLoad on navigation, the page does not automatically re-run access control here; this leaves users on an authenticated route with a logged-out session until a manual refresh/navigation happens. Align this path with other logout handlers by awaiting logout and explicitly routing to /login.
Useful? React with 👍 / 👎.
Context
Screenshots
Steps to verify the change
Type
Checklist
type(scope): short description(scope is optional, e.g.,fix: prevent crash on syncorfix(api): handle null response).