Skip to content

fix: update PAT link to redirect directly to GitHub token generation …#307

Open
harshgupta2125 wants to merge 1 commit into
GitMetricsLab:mainfrom
harshgupta2125:fix/pat-redirect-url
Open

fix: update PAT link to redirect directly to GitHub token generation …#307
harshgupta2125 wants to merge 1 commit into
GitMetricsLab:mainfrom
harshgupta2125:fix/pat-redirect-url

Conversation

@harshgupta2125
Copy link
Copy Markdown

@harshgupta2125 harshgupta2125 commented May 17, 2026

…page

Related Issue


Description

Updated the "How to generate?" link under the Personal Access Token input. Previously, it redirected users to a lengthy GitHub documentation page, causing UX friction for users trying to quickly bypass the 60-request unauthenticated API rate limit. This PR changes the href to point directly to https://github.com/settings/tokens/new, allowing users to instantly generate a token and seamlessly continue using the app.


How Has This Been Tested?

  • Ran the application locally.
  • Clicked the "How to generate?" link on the Tracker page.
  • Verified that it correctly opens the direct GitHub token creation page in a new tab rather than the documentation.

Screenshots (if applicable)

N/A — simple URL redirect update.


Type of Change

  • Bug fix
  • New feature
  • Code style update
  • Breaking change
  • Documentation update

Summary by CodeRabbit

  • Bug Fixes
    • Updated the GitHub Personal Access Token helper link in the authentication form to direct users to the correct GitHub page for token creation.

Review Change Stack

@netlify
Copy link
Copy Markdown

netlify Bot commented May 17, 2026

Deploy Preview for github-spy ready!

Name Link
🔨 Latest commit 796d7e1
🔍 Latest deploy log https://app.netlify.com/projects/github-spy/deploys/6a09f5b0778a54000834eddb
😎 Deploy Preview https://deploy-preview-307--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 17, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f6b599f5-7c52-4d8e-a5d3-f1a76d9f5179

📥 Commits

Reviewing files that changed from the base of the PR and between 8d17610 and 796d7e1.

📒 Files selected for processing (1)
  • src/pages/Tracker/Tracker.tsx

📝 Walkthrough

Walkthrough

The authentication form's GitHub Personal Access Token helper link is updated to redirect directly to the GitHub token creation page (settings/tokens/new) instead of the documentation page, improving the onboarding flow by reducing friction.

Changes

GitHub Token Helper Link Update

Layer / File(s) Summary
PAT helper link URL
src/pages/Tracker/Tracker.tsx
The Link component's href is updated from the GitHub documentation URL to the direct token creation page URL for faster token generation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

quality:clean, level:beginner

Poem

🐰 A token link, once long and wide,
Now swift as whiskers to where tokens hide,
No docs delay the eager soul—
GitHub's tokens, straight and whole!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: updating the PAT link to redirect to GitHub's token generation page instead of documentation.
Description check ✅ Passed The description follows the required template with all key sections completed: Related Issue, Description, Testing, Type of Change. It provides clear context and testing steps.
Linked Issues check ✅ Passed The PR directly addresses issue #305 by replacing the documentation link with the direct GitHub token creation URL (https://github.com/settings/tokens/new) as requested.
Out of Scope Changes check ✅ Passed The change is narrowly scoped to updating a single URL in the Tracker component, directly aligned with issue #305 requirements with no extraneous modifications.
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.

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.

🚀 Feature Enhancement : Improve UX by redirecting the PAT link directly to the GitHub Token Generation page

1 participant