docs: Auto-translate README and Wiki - #244
Open
openaitx-system wants to merge 1 commit into
Open
Conversation
Contributor
Reviewer's GuideAdds a collapsible language selector block at the top of README.md that links to auto-translated README/Wiki pages hosted on an external service for ~20 languages. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- The language selector block hardcodes
user=Arbuzovand the project name into all links; consider parameterizing these to match the canonical repository owner/name so the links remain valid if the repo is moved or renamed. - The last link for Assamese (
lang=as) appears to have a broken closing tag (</); please fix the HTML so the anchor renders correctly and doesn’t break the surrounding markup. - Since this embeds fairly complex HTML into the README, it may be worth simplifying the structure (e.g., using a plain Markdown list of links) to avoid layout issues across different renderers and improve maintainability.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The language selector block hardcodes `user=Arbuzov` and the project name into all links; consider parameterizing these to match the canonical repository owner/name so the links remain valid if the repo is moved or renamed.
- The last link for Assamese (`lang=as`) appears to have a broken closing tag (`</`); please fix the HTML so the anchor renders correctly and doesn’t break the surrounding markup.
- Since this embeds fairly complex HTML into the README, it may be worth simplifying the structure (e.g., using a plain Markdown list of links) to avoid layout issues across different renderers and improve maintainability.
## Individual Comments
### Comment 1
<location path="README.md" line_range="27" />
<code_context>
+ | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=tr">Türkçe</a>
+ | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=vi">Tiếng Việt</a>
+ | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=id">Bahasa Indonesia</a>
+ | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=as">অসমীয়া</
+ </div>
+ </div>
</code_context>
<issue_to_address>
**issue (bug_risk):** The Assamese language link appears truncated and is missing its closing HTML tags.
Add the missing `>` and closing `</a>` so the Assamese anchor is well-formed and the link renders and functions correctly.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
| | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=tr">Türkçe</a> | ||
| | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=vi">Tiếng Việt</a> | ||
| | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=id">Bahasa Indonesia</a> | ||
| | <a href="https://openaitx.github.io/view.html?user=Arbuzov&project=home_assistant_delonghi_primadonna&lang=as">অসমীয়া</ |
Contributor
There was a problem hiding this comment.
issue (bug_risk): The Assamese language link appears truncated and is missing its closing HTML tags.
Add the missing > and closing </a> so the Assamese anchor is well-formed and the link renders and functions correctly.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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/home_assistant_delonghi_primadonna/tree/Auto_translate_README_and_Wiki
Demo links :
..
If this doesn't align with your expectations, feel free to close this PR. Thanks for your time! 🙌
Summary by Sourcery
Documentation: