Skip to content

Add new list item for phattaradit.dev#304

Merged
dtinth merged 3 commits intowonderfulsoftware:mainfrom
Nongdrream:main
Feb 16, 2026
Merged

Add new list item for phattaradit.dev#304
dtinth merged 3 commits intowonderfulsoftware:mainfrom
Nongdrream:main

Conversation

@Nongdrream
Copy link
Contributor

@Nongdrream Nongdrream commented Feb 16, 2026

Summary by CodeRabbit

  • New Features
    • Added a new webring entry for phattaradit.dev, expanding the collection of linked sites.
    • Inserted phattaradit.dev into the existing ring order (placed after kasawa.pro) to maintain continuity of the public listing.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 16, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 16, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A single new webring <li> entry for phattaradit.dev was added to index.html with data-lang="en", data-owner="Nongdrream", and a link to https://phattaradit.dev, inserted into the existing ordered ring list after kasawa.pro.

Changes

Cohort / File(s) Summary
Webring Entry Addition
index.html
Inserted one new <li> webring entry for phattaradit.dev with data-lang="en", Thai metadata present, data-owner="Nongdrream", and link https://phattaradit.dev; placed after kasawa.pro in the ordered list.

Sequence Diagram(s)

(omitted — change is a single HTML list insertion, no multi-component flow)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • dtinth

Poem

🐰 I hopped along the ring today,
A tiny site found room to stay,
Phattaradit joins with a friendly cheer,
Nongdrream's name now hopping near,
Welcome, web — let's bound and play! 🎋

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding a new webring list item for phattaradit.dev, which is concise and directly related to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

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

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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
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

🤖 Fix all issues with AI agents
In `@index.html`:
- Around line 448-450: The data-lang attribute for the list item with id
"phattaradit.dev" (data-owner "Nongdrream") must be a single value, not "en,th";
update the data-lang to the site's primary language (either "en" or "th") by
changing the attribute on the <li id="phattaradit.dev"> element so it matches
other entries (e.g., data-lang="en" if the site is primarily English).

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@dtinth dtinth added the review Triggers a review label Feb 16, 2026
@github-actions github-actions bot removed the review Triggers a review label Feb 16, 2026
@github-actions
Copy link

PR validation result

Backlink:

Site description:

  • ✅ Phattaradit Muanmontree — The Cute SysAdmin. Computer Engineering student at RMUTP specializing in web development, system administration, and networking. Silver & Gold medalist in national coding competitions.

Screenshot:

Copy link
Member

@dtinth dtinth left a comment

Choose a reason for hiding this comment

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

@Nongdrream Thanks, and welcome to the webring!

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 16, 2026
@dtinth dtinth merged commit c1bd2be into wonderfulsoftware:main Feb 16, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants