Skip to content

Comments

docs: Auto-translate README and Wiki#79

Open
openaitx-system wants to merge 1 commit intotgotwig:mainfrom
openaitx-system:Auto_translate_README_and_Wiki
Open

docs: Auto-translate README and Wiki#79
openaitx-system wants to merge 1 commit intotgotwig:mainfrom
openaitx-system:Auto_translate_README_and_Wiki

Conversation

@openaitx-system
Copy link

@openaitx-system openaitx-system commented Jan 4, 2026

Added language badges to the README for easier access to translated versions: German, Spanish, French, Japanese, Korean, Portuguese, and Russian 20 languages.
System will auto-update translation for README and Wiki when this repository updated, and support multiple languages google/bing SEO search, and it's open source project, we can change web or local file mode.

The updated links can be previewed in my forked repository: https://github.com/openaitx-system/vidmerger/tree/Auto_translate_README_and_Wiki

Demo links :

Image

If this doesn't align with your expectations, feel free to close this PR. Thanks for your time! 🙌

Summary by CodeRabbit

  • Documentation
    • Added a collapsible Language section to the README with support for multiple languages.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 4, 2026

📝 Walkthrough

Walkthrough

Added a collapsible "Language" section to README.md containing language links, implemented as a right-aligned HTML block with appropriate closing tags and comments for reference.

Changes

Cohort / File(s) Summary
Documentation
README.md
Added collapsible HTML section with language links; right-aligned, includes closing placeholder and reference comment

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A language section hops into view,
With collapsible charm in HTML hue,
No logic to tangle, no bugs to pursue,
Just docs getting prettier—hop-hop, we're through!

Pre-merge checks

✅ 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 accurately describes the main change: adding language badges and an auto-translation system to the README, which directly matches the changeset content.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 044173a and 456259f.

📒 Files selected for processing (1)
  • README.md
🔇 Additional comments (2)
README.md (2)

2-31: Verify external translation service dependency.

All language links redirect to openaitx.github.io/view.html, creating a hard dependency on an external third-party service for translations. Consider:

  • Will translations be maintained if the external service becomes unavailable?
  • Is there any fallback if the service changes its URL structure or shuts down?
  • Should this be documented in the README or CONTRIBUTING guide?

If this is an intentional, documented strategy (e.g., maintained by a trusted community partner), the approach is reasonable. Otherwise, consider whether hosting translated versions or using alternative solutions would better serve the project's long-term stability.


1-31: Language section concept is well-structured.

The use of <details>/<summary> for a collapsible language selector is a good UX pattern—it keeps the README clean while providing easy access to translations across 20 languages. Once the HTML malformation is fixed, this addition should work well.


<div align="right">
<details>
<summary >🌐 Language</summary>
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Remove extra whitespace in summary tag.

There is unnecessary whitespace before the closing > in the <summary> tag.

🔎 Proposed fix
-    <summary >🌐 Language</summary>
+    <summary>🌐 Language</summary>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<summary >🌐 Language</summary>
<summary>🌐 Language</summary>
🤖 Prompt for AI Agents
In README.md around line 4, the <summary> tag contains an extra space before the
closing '>' ("<summary >..."), remove that whitespace so the tag reads
"<summary>🌐 Language</summary>" to ensure proper HTML/markdown rendering;
update the single tag on line 4 accordingly.

| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=tr">Türkçe</a>
| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=vi">Tiếng Việt</a>
| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=id">Bahasa Indonesia</a>
| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=as">অসমীয়া</
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🔴 Critical

Fix incomplete HTML tag for Assamese language link.

The closing </a> tag is missing on the final language link. The line currently ends with অসমीয়া</ instead of অসমীय়া</a>.

🔎 Proposed fix
-        | <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=as">অসমীয়া</
+        | <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=as">অসমীয়া</a>
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=as">অসমীয়া</
| <a href="https://openaitx.github.io/view.html?user=tgotwig&project=vidmerger&lang=as">অসমীয়া</a>
🤖 Prompt for AI Agents
In README.md around line 27, the Assamese language link has an incomplete HTML
anchor: the text ends with "অসমীয়া</" and is missing the closing </a> (and the
visible text appears garbled). Fix by replacing that fragment with the proper
anchor text and closing tag (e.g., use "অসমীয়া</a>") so the link text is
correct and the anchor is properly closed.

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