DanielCarter-stack commented on PR #10351: URL: https://github.com/apache/seatunnel/pull/10351#issuecomment-3759103181
<!-- code-pr-reviewer --> - **docs/** (directory structure): Breaking URL change from `connector-v2` to `connectors` will break all existing external links (blogs, tutorials, Stack Overflow answers) without a migration plan. Consider adding 301 redirects or a migration guide to preserve SEO and user bookmarks. - **docs/en/connectors/overview.md** (deleted file): The entire 433-line overview file was removed without indicating where the content migrated. If the connector capability matrix was deleted without replacement, users will lose important context about SeaTunnel's connector ecosystem. - **docs/sidebars.js**: Navigation config changed from `autogenerated` to explicit paths (135 lines changed). Without a before/after comparison, some documentation pages may become orphaned and inaccessible via the sidebar. - **docs/zh/**: Large-scale updates to both English and Chinese docs lack verification that Chinese content accurately reflects English changes. Inconsistent translations could mislead Chinese-speaking users. - **docs/** (internal links): After restructuring directories (`connector-v2` → `connectors`, moving `common-options`), internal relative links may be broken. Running a link checker before merge would prevent 404 errors within the documentation. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
