Skip to content

Conversation

@toonshouin
Copy link
Contributor

@toonshouin toonshouin commented Nov 14, 2025

Description

Base on my recent update on my site status (toonshouin/site-status#539), I moved my main site (toonstorytime.me) to (toonshou.in) due to price increases. So I want to update the webring to a current status.

Summary by CodeRabbit

  • Chores
    • Updated webring entry: replaced one member site with another. The webring list remains functionally intact with no changes to overall structure or accessibility.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Nov 14, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 14, 2025

Walkthrough

A single webring entry in index.html is replaced: the toonstorytime.me site (owner toonnongaeoy) is substituted with toonshou.in (owner toonshouin). The rest of the webring structure and all other entries remain intact.

Changes

Cohort / File(s) Summary
Webring entry replacement
index.html
Replaced webring entry from toonstorytime.me to toonshou.in with corresponding owner update from toonnongaeoy to toonshouin

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • PR #275: Makes an identical type of edit in index.html, updating a single webring list entry by replacing one site's id/href/text with a new domain/owner.

Suggested labels

size:XS, lgtm

Suggested reviewers

  • dtinth

Poem

A webring twirls, a link takes flight ✨
Old toons fade, new ones shine so bright
From toonstorytime to toonshou's door,
The circle spins, forever more! 🐰

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and directly describes the main change: updating a webring entry's domain from toonstorytime.me to toonshou.in, which aligns with the file modifications shown in the raw summary.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c07fbc9 and 1cd4a3f.

📒 Files selected for processing (1)
  • index.html (1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
index.html

📄 CodeRabbit inference engine (CLAUDE.md)

index.html: Remove identified entries from index.html by deleting the entire

  • elements for each site during cleanup
    Verify PRs add a single
  • element at the end of the list in index.html, with correct data-lang, id, data-owner, and link text
    Check that data-lang attribute is either 'en' or 'th' in the added
  • element
    Confirm id attribute matches the domain name (without subdomain if present) in the added
  • element
    Verify data-owner attribute matches the PR author's GitHub username in the added
  • element
    Ensure link text matches the id attribute in the added
  • element

  • Files:

    • index.html
    🔇 Additional comments (1)
    index.html (1)

    304-306: Entry placement does not follow guidelines—should be added at the end of the list.

    The new toonshou.in entry is correctly formatted with valid attributes (data-lang, id, data-owner, and link text all match the guidelines), but it is placed in the middle of the list (after nattawatt.com and before nokkaew.com), rather than at the end as specified in the coding guidelines.

    According to the guidelines, a single <li> element should be added at the end of the list in index.html. The last entry currently is knot-dev.me (lines 421–423), before the closing </ol> tag.

    Please verify that:

    1. The old toonstorytime.me entry was properly removed from its original location.
    2. The new entry should be repositioned to the end of the list (after knot-dev.me and before the closing </ol> tag on line 424).

    If this PR is updating an existing entry in place rather than adding a new one, please clarify the intent. Otherwise, apply this diff:

            <li data-lang="th" id="knot-dev.me" data-owner="Notties">
              <a href="https://www.knot-dev.me">knot-dev.me</a>
            </li>
    +       <li data-lang="en" id="toonshou.in" data-owner="toonshouin">
    +         <a href="https://toonshou.in">toonshou.in</a>
    +       </li>
          </ol>

    And remove the entry at lines 304–306.


    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.

    @dtinth dtinth added the review Triggers a review label Nov 19, 2025
    @github-actions github-actions bot removed the review Triggers a review label Nov 19, 2025
    @github-actions
    Copy link

    PR validation result

    Backlink:

    Site description:

    • ✅ My personal site!

    Screenshot:

    @dtinth
    Copy link
    Member

    dtinth commented Nov 19, 2025

    @toonshouin thank you so much for keeping the web link updated!

    @dtinth dtinth merged commit 248a97f into wonderfulsoftware:main Nov 19, 2025
    10 checks passed
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    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