Skip to content

Added homepage navigation option to login and signup pages#383

Open
JheelPashine123 wants to merge 2 commits into
GitMetricsLab:mainfrom
JheelPashine123:add-home-navigation-auth-pages
Open

Added homepage navigation option to login and signup pages#383
JheelPashine123 wants to merge 2 commits into
GitMetricsLab:mainfrom
JheelPashine123:add-home-navigation-auth-pages

Conversation

@JheelPashine123
Copy link
Copy Markdown

@JheelPashine123 JheelPashine123 commented May 21, 2026

Description

Implemented navigation improvements for authentication pages.

Changes Made

  • Added navigation option to return to the Home page
  • Improved user flow on Login page
  • Improved user flow on Signup page
  • Reduced dependency on browser back navigation

UI Improvements

  • Added user-friendly homepage navigation
  • Improved accessibility and UX consistency

Fixes #382

Screenshot 2026-05-22 at 2 32 06 AM

Summary by CodeRabbit

New Features

  • Added "Back to dashboard" navigation links on login and signup pages, enabling users to quickly return to the main dashboard.

Review Change Stack

@netlify
Copy link
Copy Markdown

netlify Bot commented May 21, 2026

Deploy Preview for github-spy ready!

Name Link
🔨 Latest commit 9fe9abe
🔍 Latest deploy log https://app.netlify.com/projects/github-spy/deploys/6a0f7636e9d86500082cb0e6
😎 Deploy Preview https://deploy-preview-383--github-spy.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 21, 2026

Warning

Rate limit exceeded

@JheelPashine123 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 46 minutes and 1 second before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0ffb9aa3-7b7f-4dbf-9db8-94b6c01edb86

📥 Commits

Reviewing files that changed from the base of the PR and between 04ed504 and 9fe9abe.

📒 Files selected for processing (1)
  • src/pages/Login/Login.tsx
📝 Walkthrough

Walkthrough

This PR adds a consistent navigation affordance to the authentication pages. Both the Login and Signup pages now display a "Back to dashboard" link in the top-right corner that routes users to /track, positioned above existing form content.

Changes

Auth page navigation

Layer / File(s) Summary
Back to dashboard navigation links
src/pages/Login/Login.tsx, src/pages/Signup/Signup.tsx
Adds identical right-aligned "Back to dashboard" navigation links pointing to /track on both Login and Signup pages, positioned above the existing form sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

Suggested labels

level:intermediate, quality:clean, type:accessibility

Poem

🐰 A hop back to the dashboard fair,
From login and signup with utmost care,
Two paths now lead the weary soul,
Back to /track and the metrics goal! 📊

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Added homepage navigation option to login and signup pages' accurately describes the main change—adding a navigation control to both authentication pages.
Description check ✅ Passed The description includes key sections and covers the changes made, but deviates from the template structure. Missing 'How Has This Been Tested?' and 'Type of Change' sections from the template.
Linked Issues check ✅ Passed The PR successfully implements all requirements from issue #382: adds a 'Back to dashboard' navigation link to both Login and Signup pages, reducing reliance on browser back navigation.
Out of Scope Changes check ✅ Passed All changes are scoped to adding navigation links on authentication pages as specified in issue #382. No unrelated modifications to form logic, validation, or API handling are present.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/pages/Login/Login.tsx`:
- Around line 67-76: The "Back to dashboard" Link in the Login component
currently navigates to "/track"; update the Link's to prop in
src/pages/Login/Login.tsx (the Link element inside the Login component) from
"/track" to "/" so the CTA goes to Home, and also scan the Signup component for
the same Link/copy and make the identical change if present.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: cd121130-3fd7-42da-8dd0-3e018f0856a3

📥 Commits

Reviewing files that changed from the base of the PR and between 9d34c19 and 04ed504.

📒 Files selected for processing (2)
  • src/pages/Login/Login.tsx
  • src/pages/Signup/Signup.tsx

Comment thread src/pages/Login/Login.tsx
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add navigation option to return to Home page from Login and Signup pages

1 participant