Skip to content

Remove roadmap page#108

Open
s-adamantine wants to merge 1 commit intomainfrom
remove-roadmap-page
Open

Remove roadmap page#108
s-adamantine wants to merge 1 commit intomainfrom
remove-roadmap-page

Conversation

@s-adamantine
Copy link
Copy Markdown
Contributor

@s-adamantine s-adamantine commented Mar 25, 2026 •
edited by coderabbitai bot
Loading

Summary

I noticed that this is outdated so I'm removing it for now

  • Removes the roadmap page (pages/roadmap.md) from the documentation site
  • Cleans up all references to the roadmap in navigation, homepage, FAQ, certified-identity page, search index, and metadata

Test plan

  • Verify the site builds without errors
  • Confirm no broken links to /roadmap remain
  • Check navigation renders correctly without the roadmap entry

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Removed the Roadmap page from the documentation site
    • Removed Roadmap navigation links and references from related pages and search index

@s-adamantine @claude
Remove roadmap page
47be626 
The roadmap is being removed for now. Deletes the page and cleans up
all references in navigation, index, FAQ, certified-identity docs,
search index, and lastUpdated metadata.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 25, 2026 •
edited
Loading

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

Project Deployment Actions Updated (UTC)
hypercerts-atproto-documentation Ready Ready Preview, Comment Mar 25, 2026 4:11am
hypercerts-v0.2-documentation Ready Ready Preview, Comment Mar 25, 2026 4:11am

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5e734ac9-c0a3-4e51-ba70-c440b0299a78

📥 Commits

Reviewing files that changed from the base of the PR and between 3a987ee and 47be626.

📒 Files selected for processing (8)
  • lib/generate-search-index.js
  • lib/lastUpdated.json
  • lib/navigation.js
  • pages/core-concepts/certified-identity.md
  • pages/index.md
  • pages/reference/faq.md
  • pages/roadmap.md
  • public/search-index.json
💤 Files with no reviewable changes (6)
  • pages/reference/faq.md
  • lib/lastUpdated.json
  • pages/index.md
  • lib/generate-search-index.js
  • lib/navigation.js
  • pages/roadmap.md
📝 Walkthrough

Walkthrough

The pull request removes the Roadmap page from the documentation site by deleting the roadmap.md file, eliminating all navigation references across configuration files, and removing Roadmap cross-references from other documentation pages. Related index and metadata files are updated accordingly.

Changes

Cohort / File(s) Summary
Roadmap Page Deletion
pages/roadmap.md		 Entire roadmap page file removed (343 lines of infrastructure overview, build priorities, component status, phased development plans, and related content).
Navigation & Indexing Configuration
lib/generate-search-index.js, lib/navigation.js, lib/lastUpdated.json		 Removed roadmap-specific entries from navigation configuration, search generation logic, and last-updated tracking to prevent orphaned references in generated content.
Search Index Update
public/search-index.json		 Removed complete roadmap page entry and updated certified-identity and FAQ pages to remove inline roadmap references from their indexed content.
Documentation Cross-References
pages/index.md, pages/reference/faq.md, pages/core-concepts/certified-identity.md		 Removed roadmap links and cross-references from the homepage reference section, FAQ help section, and wallet linkage documentation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 The roadmap's rolled away with care,
No longer listed anywhere,
Navigation paths run smooth and clean,
The finest docs you've ever seen! ✨
The warren bounces with pure delight! 🎉

🚥 Pre-merge checks | ✅ 3
✅ 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 accurately summarizes the main change: removing the roadmap page from the documentation site.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 remove-roadmap-page

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.

@s-adamantine s-adamantine requested a review from holkexyz March 25, 2026 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@holkexyz holkexyz Awaiting requested review from holkexyz

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@s-adamantine