Skip to content
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

docs(server-actions-and-mutations): switch onClick to formAction #64422

Closed
wants to merge 3 commits into from
Closed

docs(server-actions-and-mutations): switch onClick to formAction #64422

wants to merge 3 commits into from

Conversation

samcx
Copy link
Member

@samcx samcx commented Apr 12, 2024

@samcx samcx self-assigned this Apr 12, 2024
@samcx samcx requested review from a team as code owners April 12, 2024 22:05
@samcx samcx requested review from ismaelrumzan and StephDietz and removed request for a team April 12, 2024 22:05
@@ -946,7 +945,7 @@ export default function Page() {
...
}

return <button onClick={publish}>Publish</button>;
return <button formAction={publish}>Publish</button>;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to wrap this button in a form for this to work.

@balazsorban44 balazsorban44 added Documentation Related to Next.js' official documentation. and removed area: documentation labels Apr 17, 2024
@samcx samcx closed this by deleting the head repository Apr 23, 2024
@samcx
Copy link
Member Author

samcx commented Apr 29, 2024

Will re-open this again (deleted my fork).

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js team PRs by the Next.js team. Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants