docs: reorder portability page and tighten PDS switching section#74
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThree files updated to rebrand "Portability & Scaling" to "Portability & Data Access" in navigation, index, and documentation. Documentation page undergoes substantial restructuring to emphasize data access semantics and migration workflows while removing detailed scalability sections. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
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)
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 |
8ec0d7e to
1b42659
Compare
…ability & Data Access
1b42659 to
20092ec
Compare
There was a problem hiding this comment.
🧹 Nitpick comments (1)
pages/index.md (1)
86-87: Consider aligning the card subtitle with the new “Data Access” framing.Line 87 still emphasizes “performance,” which reads closer to the old “Scaling” label. A small wording update would keep homepage messaging consistent.
✏️ Suggested copy tweak
-Migration, performance, and privacy +Migration, app switching, and privacy🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@pages/index.md` around lines 86 - 87, Update the card subtitle inside the {% card-link %} for "Portability & Data Access": replace the current subtitle text "Migration, performance, and privacy" with wording that aligns to the new “Data Access” framing (e.g., "Migration, data access, and privacy" or similar). Locate the {% card-link title="Portability & Data Access" %} block and edit the inner text content to the chosen revised subtitle so the homepage messaging is consistent with the new label.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@pages/index.md`:
- Around line 86-87: Update the card subtitle inside the {% card-link %} for
"Portability & Data Access": replace the current subtitle text "Migration,
performance, and privacy" with wording that aligns to the new “Data Access”
framing (e.g., "Migration, data access, and privacy" or similar). Locate the {%
card-link title="Portability & Data Access" %} block and edit the inner text
content to the chosen revised subtitle so the homepage messaging is consistent
with the new label.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: cbf32672-73f2-4407-89af-93330dde2096
📒 Files selected for processing (3)
lib/navigation.jspages/architecture/portability-and-scaling.mdpages/index.md
Link to the /health endpoint on ePDS instances that support it (currently only epds1.test.certified.app, following ePDS PR #74). Production and staging instances are marked as old/unversioned with an upgrade imminent. Vanilla PDS instances (pds1) are marked n/a since the health endpoint is an ePDS-specific feature.
- Split single "Version" column into "ePDS version" and "PDS version" - ePDS version links to /health (ePDS-specific, from ePDS PR #74) - PDS version links to /xrpc/_health (standard AT Protocol endpoint) - Use footnote superscripts to note that prod/staging ePDS instances are old/unversioned, and that missing PDS version on ePDS instances is a known bug with an imminent fix - Link to ePDS release notes on GitHub from the version subsection - Prefix health links with "see" for readability
Summary
Follow-up to #71.
Fixes HYPER-162
Summary by CodeRabbit