Skip to content

[TASK-15491] fix: dont redirect to setup if user is not logged in#1268

Merged
jjramirezn merged 1 commit intopeanut-wallet-devfrom
fix/redirect-to-setup
Oct 2, 2025
Merged

[TASK-15491] fix: dont redirect to setup if user is not logged in#1268
jjramirezn merged 1 commit intopeanut-wallet-devfrom
fix/redirect-to-setup

Conversation

@jjramirezn
Copy link
Contributor

No description provided.

@jjramirezn jjramirezn requested a review from Hugo0 October 1, 2025 18:00
@vercel
Copy link

vercel bot commented Oct 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
peanut-wallet Ready Ready Preview Comment Oct 1, 2025 6:06pm

@notion-workspace
Copy link

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 1, 2025

Walkthrough

Removes authentication redirect logic from the mobile UI layout by deleting useRouter import, its initialization, and the useEffect that pushed unauthenticated users to /setup.

Changes

Cohort / File(s) Summary of changes
Auth redirect removal
src/app/(mobile-ui)/layout.tsx
Deleted useRouter import and instance; removed useEffect that redirected unauthenticated users to /setup.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • Zishan-7

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning There is no description provided for the pull request, so there is no contextual information about the change or its motivation, making it impossible to understand the scope or reasoning from the author’s notes. Please add a brief description summarizing why the redirect behavior was removed and how this change impacts the user experience, so reviewers have the necessary context.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title “[TASK-15491] fix: dont redirect to setup if user is not logged in” succinctly and accurately describes the main change by directly referencing the removal of the automatic redirect behavior for unauthenticated users, which aligns with the modifications in the layout.tsx file.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/redirect-to-setup

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 05889a1 and efd99d0.

📒 Files selected for processing (1)
  • src/app/(mobile-ui)/layout.tsx (0 hunks)
💤 Files with no reviewable changes (1)
  • src/app/(mobile-ui)/layout.tsx
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: Deploy-Preview

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jjramirezn jjramirezn merged commit 01ee131 into peanut-wallet-dev Oct 2, 2025
5 checks passed
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.

2 participants