Skip to content

fix: avoid redirect on public routes when not authenticated #232

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: avoid redirect on public routes when not authenticated #232

wants to merge 1 commit into from

Conversation

@japint
Copy link
Collaborator

@japint japint commented Jun 15, 2025

Note

🛠️ What changed

  • Prevented redirect to / when unauthenticated users visit public pages (/, /explore, /blog)
  • Moved publicPaths outside the component to clean up useEffect dependencies
  • Used window.location.pathname to check the current route

❓ Why

  • Users were being redirected from public pages even when no login was required. This fixes that.

@japint japint added the enhancement New feature or request label Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Antonov548 Antonov548 Awaiting requested review from Antonov548

@ls-ramos ls-ramos Awaiting requested review from ls-ramos

@sonali582000 sonali582000 Awaiting requested review from sonali582000

@Navya-Bharathan Navya-Bharathan Awaiting requested review from Navya-Bharathan

@savitha14june savitha14june Awaiting requested review from savitha14june

@Bianca-Dascalescu Bianca-Dascalescu Awaiting requested review from Bianca-Dascalescu

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

enhancement New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@japint