Skip to content

docs(ensrainbow): add tip aside for ENSRainbow landing page#2257

Merged
lightwalker-eth merged 2 commits into
mainfrom
add-ensrainbow-link
Jun 5, 2026
Merged

docs(ensrainbow): add tip aside for ENSRainbow landing page#2257
lightwalker-eth merged 2 commits into
mainfrom
add-ensrainbow-link

Conversation

@djstrong
Copy link
Copy Markdown
Member

@djstrong djstrong commented Jun 4, 2026

Lite PR

Tip: Review docs on the ENSNode PR process

Summary

  • What changed (1-3 bullets, no essays).

Why

  • Why this change exists. Link to related GitHub issues where relevant.

Testing

  • How this was tested.
  • If you didn't test it, say why.

Notes for Reviewer (Optional)

  • Anything non-obvious or worth a heads-up.

Pre-Review Checklist (Blocking)

  • This PR does not introduce significant changes and is low-risk to review quickly.
  • Relevant changesets are included (or are not required)

Copilot AI review requested due to automatic review settings June 4, 2026 19:13
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 4, 2026

⚠️ No Changeset found

Latest commit: 5b31527

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Jun 4, 2026

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

Project Deployment Actions Updated (UTC)
enskit-react-example.ensnode.io Ready Ready Preview, Comment Jun 4, 2026 7:52pm
ensnode.io Ready Ready Preview, Comment Jun 4, 2026 7:52pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
admin.ensnode.io Skipped Skipped Jun 4, 2026 7:52pm
ensrainbow.io Skipped Skipped Jun 4, 2026 7:52pm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a Starlight “tip” aside to the ENSRainbow docs overview page to help readers discover the project’s external landing page.

Changes:

  • Import Aside from @astrojs/starlight/components.
  • Add a tip <Aside> linking to https://ensrainbow.io near the top of the ENSRainbow overview doc.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 4, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This PR updates the ENSRainbow services documentation page by importing the Aside component and adding a tip callout that directs users to the ENSRainbow landing page at ensrainbow.io.

Changes

ENSRainbow Documentation Update

Layer / File(s) Summary
Add landing page tip callout
docs/ensnode.io/src/content/docs/docs/services/ensrainbow/index.mdx
Import Aside component and add a tip callout asking "Looking for the ENSRainbow landing page?" with a link to ensrainbow.io.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

docs

Poem

🐰 A tip for wanderers lost in the docs,
Where ENSRainbow lives atop the rocks,
We point the way with a gentle nudge,
To ensrainbow.io—a helpful fudge! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description uses the correct template structure but all required sections lack substantive content—Summary, Why, Testing, and Notes are unfilled placeholders. Fill in the Summary section with 1-3 bullets describing what changed (e.g., added Aside component import and tip callout). Complete the Why section with rationale and any related issue links. Complete the Testing section with testing details or justification for lack of testing.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a tip aside component for the ENSRainbow landing page link in documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch add-ensrainbow-link

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.

@vercel vercel Bot temporarily deployed to Preview – admin.ensnode.io June 4, 2026 19:50 Inactive
@vercel vercel Bot temporarily deployed to Preview – ensrainbow.io June 4, 2026 19:50 Inactive
@djstrong djstrong marked this pull request as ready for review June 4, 2026 19:54
@djstrong djstrong requested a review from a team as a code owner June 4, 2026 19:54
Copilot AI review requested due to automatic review settings June 4, 2026 19:54
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jun 4, 2026

Greptile Summary

Adds a Starlight <Aside type="tip"> component to the ENSRainbow landing page, directing new readers to ensrainbow.io for a high-level introduction before they dive into the technical documentation.

  • Imports and uses Aside from @astrojs/starlight/components alongside the existing LinkCard import — no structural changes to the rest of the page.
  • The aside is placed after the two introductory paragraphs and before the main content sections, which is a natural discovery point for new visitors.

Confidence Score: 5/5

Documentation-only change with no functional code impact; safe to merge.

A single Aside tip component is added to a Markdown/MDX doc page. The import is correct, the component usage matches Starlight's API, and the link target (ensrainbow.io) is a legitimate external site. Nothing here can break a build or affect runtime behavior.

No files require special attention.

Important Files Changed

Filename Overview
docs/ensnode.io/src/content/docs/docs/services/ensrainbow/index.mdx Adds an Aside tip component linking to ensrainbow.io; import and usage are correct, placement is appropriate, no issues found.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Visitor lands on ENSRainbow docs page] --> B[Reads intro paragraphs]
    B --> C{New to ENSRainbow?}
    C -- Yes --> D["<Aside type='tip'>\nVisit ensrainbow.io\nfor high-level intro"]
    C -- No --> E[Continue to technical sections]
    D --> F[ensrainbow.io]
    D --> E
Loading

Reviews (1): Last reviewed commit: "docs(ensrainbow): update tip aside for c..." | Re-trigger Greptile

Copy link
Copy Markdown
Member

@lightwalker-eth lightwalker-eth left a comment

Choose a reason for hiding this comment

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

@djstrong Looks good, thanks 👍

@lightwalker-eth lightwalker-eth merged commit 6542aa4 into main Jun 5, 2026
24 checks passed
@lightwalker-eth lightwalker-eth deleted the add-ensrainbow-link branch June 5, 2026 10:28
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.

3 participants