Skip to content

Conversation

@react-translations-bot
Copy link
Collaborator

This PR was automatically generated.

Merge changes from react.dev at fcd0006

DO NOT SQUASH MERGE THIS PULL REQUEST!

Doing so will "erase" the commits from main and cause them to show up as conflicts the next time we merge.

brianjo and others added 18 commits October 30, 2023 17:43
* Adds landing page for reference section.)

* Updated hyperlinks.

* Fixes Legacy APIs link.

* Removes canary image from heading.
Co-authored-by: Sophie Alpert <git@sophiebits.com>
Co-authored-by: Luna Wei <luwe@fb.com>
Co-authored-by: Soichiro Miki <smiki-tky@umin.ac.jp>
Co-authored-by: David McCabe <davemccabe@gmail.com>
Co-authored-by: Luna Wei <luwe@fb.com>
* Edits for new "use client" content

* Apply suggestions from code review

Co-authored-by: Luna <lunaleaps@gmail.com>

---------

Co-authored-by: Luna <lunaleaps@gmail.com>
Summary of changes:
"Server Action", like "Effect", is a React-specific notion
that would be benefited from captailization to be distinguished
from its genertic meaning.

It seems like [Next doc](https://nextjs.org/docs/app/api-reference/functions/server-actions)
has also adopted such connventions and we should probably do the same.

Co-authored-by: xuan.huang <xuan.huang@bytedance.com>
Summary of changes:
following <reactjs/react.dev#6417>

Co-authored-by: xuan.huang <xuan.huang@bytedance.com>
Summary of changes:
See <reactjs/react.dev#6417>

Co-authored-by: xuan.huang <xuan.huang@bytedance.com>
Co-authored-by: Luna Wei <luwe@fb.com>
Summary of changes:
See <reactjs/react.dev#6417>.

There is one exception though. The error message
“Rendered fewer hooks than expected” from React is indeed "hooks"
so I kept as-is. Shall we change the error message from React?

Co-authored-by: xuan.huang <xuan.huang@bytedance.com>
@TheBrown TheBrown merged commit df40aba into main Nov 16, 2023
@TheBrown TheBrown deleted the sync-fcd00068 branch November 16, 2023 00:40
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.