Repository navigation
Conversation
There was a problem hiding this comment.
Looks good. This is a docs-only change, so there was nothing to run. I checked the PR head: the site footer (services/site/src/component/site/links.ts) already links only Discord for every locale, and the footer.feishu strings aren't used anywhere. So this line brings README.zht.md in line with the site and README.md, and README.zh.md keeps Feishu as #16908 intended. The public API is unchanged.
For maintainers: #16908 chose Feishu on purpose for both zh and zht. This undoes that only for the zht README, which the site footer has in effect already done.
This branch has not been deployed
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.
Issue for this PR
Closes #27642 (that issue was closed as not-planned by the cleanup bot while the fix was pending review; this re-opens just the surviving piece).
Type of change
What does this PR do?
README.zht.md still sends Traditional Chinese readers to the Feishu group, while the English README already links Discord. The broader fix (#27640 by @changchinlan, later ported as #54593) was auto-closed by the PR cleanup and is now mostly moot: the V1 console was removed (#63117), the new site footer links Discord for every locale, and the 飛書 label is already corrected in
services/site/src/i18n/zht.ts. The only piece still present ondevis this one line in README.zht.md, so this PR is just that line.Feishu stays for README.zh.md on purpose (#16908) — only the zht README changes, and the new line matches the English README's format exactly.
How did you verify your code works?
Docs-only, one line. Link target
https://discord.gg/opencodeis the same one already used in README.md and the site footer config, and the surrounding community line structure is unchanged.Screenshots / recordings
N/A — link-target and text change only.
Checklist