Refactor _meta.json to rename 'slack' section to 'docs' and update ti…#470
Refactor _meta.json to rename 'slack' section to 'docs' and update ti…#470
Conversation
…tle to 'Docs' for improved clarity and consistency in documentation structure.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Signed-off-by: Vrijraj Singh <vrijraj2396@gmail.com>
📝 WalkthroughWalkthroughThis PR establishes a new integrations directory structure for app nodes by creating legacy-to-integration mappings, updating redirect destinations from deprecated node documentation paths to new integration pages, and adding legacy node slug metadata to integration documentation files. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ 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 |
- Modified permanent redirects in `next.config.mjs` to point crawler and scraper nodes to the new Firecrawl integration. - Enhanced `_meta.json` by adding an "Apps" section for better navigation. - Added `legacyNodeSlugs` to various integration files (Airtable, AWS S3, Firecrawl, Google Drive, Google Sheets, N8N, Notion, OneDrive, PostgreSQL, SharePoint, Slack, Twilio, Web Search) to maintain compatibility with previous node names. - Improved documentation clarity and structure across multiple integration files.
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
next.config.mjs (1)
107-111:⚠️ Potential issue | 🟠 Major
permanentRedirectsare served withpermanent: false— likely unintentional.The
permanentRedirectsarray is mapped withpermanent: false(Line 110), identical tononPermanentRedirects. If these are truly meant to be permanent (HTTP 308), this should bepermanent: true. This is a pre-existing issue but worth noting since this PR adds entries to thepermanentRedirectsarray.Proposed fix
...permanentRedirects.map(([source, destination]) => ({ source, destination, - permanent: false, + permanent: true, })),
🧹 Nitpick comments (1)
pages/integrations/apps-data-sources/google-drive.mdx (1)
19-26: Redundant description acrossNodeTypeInfo, the paragraph below it, and the Overview paragraph above.The
descriptionprop onNodeTypeInfo(lines 23), the paragraph on line 26, and the overview paragraph on line 17 all convey essentially the same information — that this node is a Batch Trigger for file fetching/sync and an Action node for doc/folder management. Consider removing the free-text paragraph on line 26, sinceNodeTypeInfoalready renders the description and the Overview section already covers it.Suggested diff
<NodeTypeInfo batchTrigger={true} eventTrigger={false} action={true} description="This node is a Batch Trigger node that automates file fetching and synchronization from Google Drive, and an Action node for creating, editing, and managing docs and folders." /> - -This node is a **Batch Trigger** node that streamlines file collection from Google Drive and prepares files for vectorization and indexing to enhance RAG flows. It also supports **Action** mode for direct operations on Google Drive (create doc, edit file, create folder, list files, move file, get file metadata).
…tle to 'Docs' for improved clarity and consistency in documentation structure.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation