Skip to content

feat: remove roadmap page and related components#5602

Open
4bayy wants to merge 7 commits into
asyncapi:masterfrom
4bayy:feat-Remove-the-Roadmap-from-the-website-#5572
Open

feat: remove roadmap page and related components#5602
4bayy wants to merge 7 commits into
asyncapi:masterfrom
4bayy:feat-Remove-the-Roadmap-from-the-website-#5572

Conversation

@4bayy

@4bayy 4bayy commented Jun 28, 2026

Copy link
Copy Markdown

Description

This PR removes the roadmap page and its related components from the website.

Changes

  • Removed the roadmap page.
  • Removed roadmap-related components.
  • Removed the associated Cypress tests.
  • Updated related configuration and navigation.

Testing

  • ✅ Production build completed successfully (npm run build).
  • ✅ Verified the application works after removing the roadmap feature.

Related Issue

Closes #5572

Summary by CodeRabbit

  • Changes
    • Removed the Roadmap page and its associated UI from the site.
    • Updated navigation to hide the Roadmap link, so it’s no longer accessible from the homepage.
    • Kept the remaining navigation items unchanged.
  • Tests
    • Removed Roadmap-related end-to-end coverage, including the Roadmap page checks and navigation test flows.

@netlify

netlify Bot commented Jun 28, 2026

Copy link
Copy Markdown

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 1c24e9d
🔍 Latest deploy log https://app.netlify.com/projects/asyncapi-website/deploys/6a553d76948fb000082fa7c8
😎 Deploy Preview https://deploy-preview-5602--asyncapi-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

The /roadmap page, its components and types, navigation link, and Cypress references are removed. The triage workflow checkout and formatting are updated, and Book icon JSDoc now documents parameter and return types.

Changes

Roadmap Removal

Layer / File(s) Summary
Roadmap page, components, and types deleted
pages/roadmap.tsx, components/roadmap/*
Removes the roadmap page, rendering components, and roadmap item type definitions.
Navigation and Cypress cleanup
components/navigation/otherItems.tsx, cypress/pages/RoadmapPage.js, cypress/pages/homepage.js, cypress/homepage.cy.js
Removes the Roadmap navigation entry, page object, navigation method, and E2E test case.

Repository Maintenance

Layer / File(s) Summary
Workflow and icon documentation updates
.github/workflows/notify-triager.yml, components/icons/Book.tsx
Updates the checkout action reference, removes trailing whitespace, and expands the Book icon JSDoc.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: derberg, akshatnema, sambhavgupta0705, anshgoyalevil, asyncapi-bot-eve

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The PR includes unrelated changes to the triager workflow and Book icon docs that are not needed to remove the Roadmap. Remove or split out the workflow and JSDoc-only edits so the PR stays focused on the Roadmap removal.
✅ 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 clearly and concisely describes the main change: removing the Roadmap page and related components.
Linked Issues check ✅ Passed The PR removes the Roadmap page, related components, navigation, and tests, matching issue #5572's request to remove the Roadmap from the website.
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

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.

@codecov

codecov Bot commented Jun 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (8d7dab1) to head (3db7a89).
⚠️ Report is 48 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #5602   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines          931       976   +45     
  Branches       180       192   +12     
=========================================
+ Hits           931       976   +45     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@asyncapi-bot

asyncapi-bot commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 46
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-5602--asyncapi-website.netlify.app/

@princerajpoot20 princerajpoot20 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@4bayy please have a look at the failing pipeline

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@components/icons/Book.tsx`:
- Around line 5-9: Insert a blank line immediately before the JSDoc comment for
the Book icon component, without changing the comment or surrounding
implementation, so the lines-around-comment ESLint rule passes.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e6a458f3-f403-4641-a693-297de90ee70b

📥 Commits

Reviewing files that changed from the base of the PR and between 49ca845 and e4e8f86.

📒 Files selected for processing (3)
  • .github/workflows/notify-triager.yml
  • components/icons/Book.tsx
  • components/navigation/otherItems.tsx
✅ Files skipped from review due to trivial changes (1)
  • components/navigation/otherItems.tsx

Comment thread components/icons/Book.tsx

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/notify-triager.yml:
- Around line 12-16: Fix the indentation of the Checkout Repository step in the
workflow so uses and with are aligned as step properties, and repository, ref,
and fetch-depth are nested beneath with. Preserve the existing checkout action
and input values.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a5d53055-eebe-4e56-b680-8e0e4d93334c

📥 Commits

Reviewing files that changed from the base of the PR and between 4558d82 and 9e40f9d.

📒 Files selected for processing (1)
  • .github/workflows/notify-triager.yml

Comment thread .github/workflows/notify-triager.yml Outdated
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: To Be Triaged

Development

Successfully merging this pull request may close these issues.

Remove the Roadmap from the website

3 participants