-
Notifications
You must be signed in to change notification settings - Fork 184
Change from supakorn.info to supakorn.me #299
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.
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
base: main
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughUpdated a webring entry domain from supakorn.info to supakorn.me in index.html, changing the list item ID, anchor href, and display text accordingly. This is a content-only modification with no logic or script changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. 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. Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Updates the webring member entry to reflect a domain migration away from supakorn.info, ensuring the public ring list points at the current site.
Changes:
- Replaced the
supakorn.infolist entry withsupakorn.mein the ring member list. - Updated associated attributes (
id,href,data-owner) for that entry.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <li data-lang="th" id="supakorn.me" data-owner="supakornn"> | ||
| <a href="https://supakorn.me">supakorn.me</a> |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing the <li> id from supakorn.info to supakorn.me changes the key used by index.js to look up siteData[id] (fetched from webring-site-data/data.json). If the external site-data JSON isn’t updated with the new key, this site will lose its metadata (description/images/etc.) in the UI. Please update the corresponding site-data source (or run the site-updater workflow) so the new id exists there.
| <li data-lang="th" id="supakorn.me" data-owner="supakornn"> | ||
| <a href="https://supakorn.me">supakorn.me</a> |
Copilot
AI
Feb 11, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR description says the new domain is supkorn.me, but this change updates the entry to supakorn.me. Please confirm which domain is correct and align the PR description and the link/id here accordingly to avoid publishing the wrong URL.
Summary by CodeRabbit