-
Notifications
You must be signed in to change notification settings - Fork 27k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove the experimental serverActions
flag
#57145
Conversation
Notifying the following users due to files changed in this PR based on this repo's notify modifiers: @timneutkens, @ijjk, @shuding, @styfle, @huozhi:
|
3a80221
to
586009a
Compare
Tests Passed |
Stats from current PRDefault BuildGeneral
Client Bundles (main, webpack)
Legacy Client Bundles (polyfills)
Client Pages
Client Build Manifests
Rendered Page Sizes
Edge SSR bundle Size
Middleware size
|
serverActions
flag
664cc6a
to
7221a46
Compare
Co-authored-by: Zack Tanner <[email protected]>
LFG |
What are you absolute cowboys doing just publishing this to your docs when this none of this is in the stable releases yet. Do you enjoy wasting people's time? |
@leonaves some React docs have already landed and more will be landing this week. Note that the |
I mean someone has already reverted this change in the live docs since yesterday so you obviously know you messed up. |
Also, React's docs say clearly the new hooks are only available in canary, whereas running:
Installs react-dom |
It was not clear to me that you meant that docs for this were live already, which then prompted me to go look, and you were correct - those slipped out. While they're merged into main here, they shouldn't have been live yet, until our release Thursday. Sorry about that. (More specifically, docs being merged into this repo shouldn't mean they go live immediately. It should be part of a release being tagged) Edit: Also -> https://twitter.com/reactjs/status/1716573234160967762 |
Remove the experimental
serverActions
flag