Skip to content

Refactor permanent redirects and update NodesIndex component for impr…#466

Merged
vrijraj merged 1 commit intomainfrom
dev
Feb 13, 2026
Merged

Refactor permanent redirects and update NodesIndex component for impr…#466
vrijraj merged 1 commit intomainfrom
dev

Conversation

@vrijraj
Copy link
Contributor

@vrijraj vrijraj commented Feb 13, 2026

…oved app integration documentation

  • Updated permanent redirects in next.config.mjs to reflect new paths for various app nodes, enhancing navigation.
  • Modified NodesIndex.tsx to fetch app data from the new integrations/apps-data-sources path, ensuring accurate categorization and display of app nodes.
  • Removed outdated app node documentation files (Airtable, Crawler, Cron, Gmail, Google Drive, Google Sheets) to streamline content and focus on current integrations.

Summary by CodeRabbit

Release Notes

  • Documentation
    • Reorganized app integration documentation to a dedicated integrations section for improved navigation and accessibility.
    • Updated navigation structure to reflect the new documentation organization.
    • Implemented URL redirects to maintain backward compatibility with previous documentation links.

…oved app integration documentation

- Updated permanent redirects in `next.config.mjs` to reflect new paths for various app nodes, enhancing navigation.
- Modified `NodesIndex.tsx` to fetch app data from the new integrations/apps-data-sources path, ensuring accurate categorization and display of app nodes.
- Removed outdated app node documentation files (Airtable, Crawler, Cron, Gmail, Google Drive, Google Sheets) to streamline content and focus on current integrations.
@vercel
Copy link

vercel bot commented Feb 13, 2026

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

Project Deployment Actions Updated (UTC)
site Error Error Feb 13, 2026 11:01am

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 13, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

This PR migrates app-related documentation from /docs/nodes/apps/ to /integrations/apps-data-sources/, updates the NodesIndex component to source Apps data from the new location, adds redirects for old paths, and removes the original documentation files from their legacy location.

Changes

Cohort / File(s) Summary
Component & Configuration Updates
components/NodesIndex.tsx, next.config.mjs, pages/docs/nodes/_meta.json
Modified NodesIndex to pull Apps from /integrations/apps-data-sources instead of /docs/nodes/apps; added 20+ redirects mapping old app paths to new locations; changed Apps metadata from type: "doc" to route: "/docs/integrations/"
Deleted App Documentation
pages/docs/nodes/apps/* (airtable-node, crawler-node, cron-node, gmail-node, google-drive-node, google-sheets-node, n8n-node, notion-node, onedrive-business-node, postgres-node, s3-node, scraper-node, sharepoint-business-node, slack-node, smtp-node, twilio-node, web-search-node)
Removed 17 MDX documentation pages previously describing app node integrations, including configuration references, examples, troubleshooting, and output schemas. Content relocated to /integrations/apps-data-sources/.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Poem

🐰 Apps pack their bags, off to a new home,
From docs/nodes to integrations they roam,
Redirects guide old paths astray,
While NodesIndex learns the new way,
Seventeen tales find their proper place—
A tidy migration, with digital grace!

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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.

@vrijraj vrijraj merged commit f38bb90 into main Feb 13, 2026
1 of 3 checks passed
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.

1 participant