Skip to content

fix: update clone urls to itsnyein after account rename - #26

Merged
itsnyein merged 1 commit into
mainfrom
fix/repo-url-casing
Aug 21, 2026
Merged

itsnyein merged 1 commit into
mainfrom
fix/repo-url-casing

Conversation

@nyeinphyoaung

Copy link
Copy Markdown
Collaborator

Summary

The GitHub account was renamed from Its-Nyein to itsnyein. GitHub redirects the old URL, but the documented clone commands still used the old casing.

Changes

  • README.md - clone URL updated to github.com/itsnyein/shadcn-admin.git
  • CONTRIBUTING.md - same

lib/github.ts also hardcoded the old owner (GITHUB_OWNER), which drives the star-count API call and the git clone command rendered on the landing page. That file only exists on feat/landing-page, so it is fixed there in #25 rather than here.

@vercel

vercel Bot commented Aug 21, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
shadcn-admindashboard Ready Ready Preview Aug 21, 2026 10:22am

@claude

claude Bot commented Aug 21, 2026 •

Copy link
Copy Markdown

Claude finished @nyeinphyoaung's task in 57s —— View job


Todo List

  • Gather context (diff, changed files)
  • Review changes against project conventions
  • Post code review

Code Review: fix: update clone URLs to itsnyein after account rename

Summary

This is a minimal, low-risk documentation fix that corrects the GitHub account casing (Its-Nyein -> itsnyein) in the clone URLs in README.md and CONTRIBUTING.md. The change is accurate, scoped, and matches its stated purpose. No code, config, or logic is touched.


Critical Issues

None.


Medium Issues

None.


Minor Issues

1. lib/github.ts follow-up not yet merged (lib/github.ts)

The PR description notes that GITHUB_OWNER in lib/github.ts (star-count API + landing page clone command) still hardcodes the old casing, and that it's fixed separately in #25/landing-page branch. That file doesn't exist on main yet in this branch's history, so there's nothing to check here — just flagging as a reminder to confirm #25 (or its successor) actually lands the same rename before/after merge, so the two don't drift.


Strengths

  • Change is exactly scoped to the stated problem — no unrelated edits.
  • Verified via grep that no other Its-Nyein references remain in the repository after this fix.
  • Clear, factual PR description explaining the GitHub rename and why lib/github.ts is deliberately excluded.

Verdict: Ready to merge.

@itsnyein itsnyein changed the title fix: update clone URLs to itsnyein after account rename fix: update clone urls to itsnyein after account rename Aug 21, 2026
@itsnyein
itsnyein merged commit 400b509 into main Aug 21, 2026
4 checks passed
@itsnyein
itsnyein deleted the fix/repo-url-casing branch August 23, 2026 09:23

This branch was successfully deployed

1 active deployment
Preview — e284c5df Deployed Aug 21, 2026 by vercel[bot]
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