Skip to content

Conversation

@docschina-bot
Copy link
Collaborator

此 PR 自动生成。

合并 reactjs/reactjs.org 的修改至本仓库(@ d271a7a

以下文件存在冲突,可能需要重新翻译:

请解决所有冲突后提交到此 pull request,可直接在 Github 修改、切换到 sync-d271a7ac 分支进行修改或者使用 Github Desktop 拉取此 pr 修改并提交。

禁止使用 Squash Merge 来合并此 Pull Request!

使用该操作会导致上游仓库的内容丢失,并致使下一次合并产生冲突。

martinrebo and others added 7 commits November 5, 2025 18:39
* fix: correct link for RedwoodJS to RedwoodSDK in documentation

* fix: update links for React Query and TanStack Start in documentation

* fix: update Vite installation command to use TypeScript template

* fix: update references from React Query to TanStack Query in documentation
Fixes #8097

The refactored example in the "Chains of computations" section uses
an incorrect condition that changes the game logic from the original.

The original Effect-based code advances the round after 4 gold cards:
- Increments first (0→1, 1→2, 2→3, 3→4)
- Then checks `goldCardCount > 3` (true when count is 4)

The refactored code with `goldCardCount <= 3` allows 5 gold cards:
- Checks before incrementing
- Allows counts 0, 1, 2, 3 to increment (4 values)
- Advances on the 5th card (when count is 4)

This fix changes the condition to `goldCardCount < 3`:
- Allows counts 0, 1, 2 to increment (3 values)
- Advances on the 4th card (when count is 3)
- Matches the original behavior

Verified by tracing execution logic and building the docs site locally.

Co-authored-by: PaulyBearCoding <PaulyBearCoding@users.noreply.github.com>
@docschina-bot docschina-bot requested a review from QC-L November 6, 2025 22:22
@vercel
Copy link

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
zh-hans-react-dev Ready Ready Preview Comment Nov 7, 2025 6:35am

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@loveloki loveloki merged commit cbf4c7e into main Nov 7, 2025
8 checks passed
@loveloki loveloki deleted the sync-d271a7ac branch November 7, 2025 06:44
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.

8 participants